-
Notifications
You must be signed in to change notification settings - Fork 43
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
Phew._handle_request does not ensure content-length of data read #56
Comments
I ran into this problem, too, but with a form with email addresses. I fixed it in a similar manner. In that case, the parser did not have a complete name=value pair, but only sometimes as the email addresses I entered changed length. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When
content-type
isapplication/x-www-form-urlencode
d, the current implementation makes a single call to read:Unfortunately, this doesn't always read all
content-length
bytes. It's necessary to loop until all bytes are read e.g.I was posting a textarea with about 1400 bytes. It was only reading about 250.
Other parts of the implementation may need to be checked to ensure that no other similar issue is present.
The text was updated successfully, but these errors were encountered: