Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only copy body request content if body is present #3368

Closed
wants to merge 1 commit into from

Conversation

benba
Copy link
Contributor

@benba benba commented Apr 22, 2024

This avoids to have a "Transfer-Encoding: chunked" with an empty body if downstream exchanges are negociated in HTTP 1.1

Fixes gh-3325

This avoids to have a "Transfer-Encoding: chunked" with an empty body if downstream exchanges are negotiated in HTTP 1.1

Fixes spring-cloudgh-3325
@cweberruss
Copy link

I would appreciate this PR since we are encountering the same problem with GET requests going through AWS ELB2 resulting in 400 - BAD REQUEST.

@spencergibb can you maybe have a look at this PR? Thanks in advance!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Gateway MVC] Transfer-Encoding added to GET/HEAD requests, causing some WAF errors
3 participants