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
Support HTTP errors 414 and 431 #2534
Conversation
414 is "uri too long" https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/414 431 is "header too long" https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/431 Old behavior was using 413 "request payload too large" which is specific to the content, not the first line or headers.
@yuzawa-san Please sign the Contributor License Agreement! Click here to manually synchronize the status of this Pull Request. See the FAQ for frequently asked questions. |
@yuzawa-san Thank you for signing the Contributor License Agreement! |
@reactor/team PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@simonbasle @pderop Thanks for the review! |
414 is "uri too long" https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/414 431 is "header too long" https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/431 Old behaviour was using 413 "request payload too large" which is specific to the content, not the first line or headers.
@yuzawa-san Thanks for the PR! |
* Bump reactor-netty to 2022.0.7 * fix flaky tests * test: fix MaxHeaderSizeTest See reactor/reactor-netty#2534 * test: fix ConnectionPoolTest * now we should expect Bad Request with capital R * also, minor refactor in some tests while trying to reproduce failings test: working on CacheContentLengthLimitTest * ci: add Surefire JUnit report collection apply some suggestions from #404 --------- Co-authored-by: paolo.venturi <paolo.venturi@diennea.com> Co-authored-by: Niccolò Maltoni <niccolo.maltoni@diennea.com>
414 is "uri too long" https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/414 431 is "header too long" https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/431 Old behavior was using 413 "request payload too large" which is specific to the content, not the first line or headers.
I added this upstream to Netty a while ago, netty/netty#12084
vert.x seems to have upgraded their implementation in a similar manner