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.

Sign up for GitHub

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

Jump to bottom

[UNDERTOW-1492] Respect rfc7230 section-3.3.2 #856

Merged
merged 1 commit into from
Mar 10, 2020
Merged

[UNDERTOW-1492] Respect rfc7230 section-3.3.2 #856

merged 1 commit into from
Mar 10, 2020

Conversation

thomasbabtist
Copy link
Contributor

Undertow send Content-Length header in 204 responses for HTTP/2, thus violating RFC 7230.
Note that for HTTP/1.X Undertow handles this correctly

@undertow-pull-request
Copy link

Can one of the admins verify this patch?

@fl4via
Copy link
Member

fl4via commented Mar 10, 2020

This is ok to test

@fl4via fl4via added bug fix Contains bug fix(es) next release This PR will be merged before next release or has already been merged (for payload double check) waiting CI check Ready to be merged but waiting for CI check and removed waiting CI check Ready to be merged but waiting for CI check labels Mar 10, 2020
@fl4via fl4via merged commit 89756ac into undertow-io:master Mar 10, 2020
@fl4via fl4via removed the next release This PR will be merged before next release or has already been merged (for payload double check) label Mar 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fl4via fl4via fl4via approved these changes

Assignees
No one assigned
Labels
bug fix Contains bug fix(es)
Projects
None yet
Milestone
No milestone
3 participants
@thomasbabtist @undertow-pull-request @fl4via