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

Don't panic if chunked encoding contains no chunk length #52

Merged

rmja merged 1 commit into drogue-iot:main from bugadani:panic

Oct 24, 2023

Contributor

bugadani commented Oct 24, 2023 •

edited

Loading

If we read a newline in place of the chunk length, total_read is 0 and the subtraction for the \r check underflows. This PR fixes this issue.


          Don't panic if chunked encoding contains no chunk length

7c090d7

rmja approved these changes

View reviewed changes

rmja merged commit 2f71cc9 into drogue-iot:main

1 check passed

bugadani deleted the panic branch

October 24, 2023 09:24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet