-
-
Notifications
You must be signed in to change notification settings - Fork 568
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
[ChunkParser] does not handle additional headers fields after the message body. #44
Comments
Thanks, @briantting for reporting this and #45 . I am wondering what exactly is your use case that runs into these issues. Can you expand a little on that? |
Hi Abhinav,
I'm trying to run a series of web-platform-tests through a http proxy to
test my code. The tests involve a lot of more unusual use cases that I've
run into.
Cheers,
Brian
…On Sun, Feb 24, 2019 at 9:53 AM Abhinav Singh ***@***.***> wrote:
Thanks, @briantting <https://github.com/briantting> for reporting this
and #45 <#45> . I am
wondering what exactly is your use case that runs into these issues. Can
you expand a little on that?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#44 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AB7Bg2t2_mo5tA3PfWFPkwGcO-Q-n-A-ks5vQtG2gaJpZM4bM5Ef>
.
|
Thanks, @briantting, good to hear that. I observed issues you reported are encountered when packets are sent to |
@briantting Is it possible for you to give it another shot and/or provide raw requests being sent to |
It gets caught in a loop when processing chunked requests where there are additional headers fields after the message body. Occurs because:
self.size
inChunkParser.process
is0
self.state
is set toCOMPLETE
, but thelen(data) > 0
The text was updated successfully, but these errors were encountered: