-
Notifications
You must be signed in to change notification settings - Fork 647
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
[Feature] Support 'allowPartialChunks' in the HttpDecoderSpec #3452
Labels
type/enhancement
A general enhancement
Milestone
Comments
reta
added
status/need-triage
A new issue that still need to be evaluated as a whole
type/enhancement
A general enhancement
labels
Oct 2, 2024
violetagg
removed
the
status/need-triage
A new issue that still need to be evaluated as a whole
label
Oct 3, 2024
@reta We love PRs :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Motivation
The Netty's
HttpDecoderConfig
exposesallowPartialChunks
property which hints theHttpObjectDecoded
if the chunk could be split into multiple when chunk exceeds the readable bytes. TheHttpDecoderSpec
sadly does not provide the way to configure it.Desired solution
Support 'allowPartialChunks' in the
HttpDecoderSpec
so it could be used to createHttpDecoderConfig
instances with respective property set.Considered alternatives
N/A
Additional context
We right now are running into an issue when the complete chunk should be consumed but the partial ones are being produced instead.
@violetagg I am happy to submit the pull request if the feature makes sense to you, thank you
The text was updated successfully, but these errors were encountered: