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

content-length header should not be required for HTTP/2 requests #8449

Closed
wtlucy opened this issue Aug 1, 2019 · 0 comments · Fixed by #8447
Closed

content-length header should not be required for HTTP/2 requests #8449

wtlucy opened this issue Aug 1, 2019 · 0 comments · Fixed by #8447
Assignees
@wtlucy
Labels
in:Transport release bug This bug is present in a released version of Open Liberty release:19008 team:Sirius

Comments

@wtlucy
Copy link
Contributor

wtlucy commented Aug 1, 2019

Currently, HTTP/2 request message/body data is ignored if no content-length header is present. Given HTTP/2's stream and framing mechanisms, a content-length is not needed to identify the size of a data payload. The HTTP channel should be updated so that HTTP/2 request bodies are usable even if no content-length is present.
@wtlucy wtlucy added in:Transport release bug This bug is present in a released version of Open Liberty team:Sirius labels Aug 1, 2019
@wtlucy wtlucy self-assigned this Aug 1, 2019
@wtlucy wtlucy mentioned this issue Aug 1, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wtlucy wtlucy

Labels
in:Transport release bug This bug is present in a released version of Open Liberty release:19008 team:Sirius
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Support HTTP/2 request bodies without content-length wtlucy/open-liberty
2 participants
@wtlucy @LibbyBot