Skip to content

Conversation

mjforan
Copy link
Contributor

@mjforan mjforan commented Aug 23, 2020

This PR fixes #7542. I rewrote the section of the Web Server library which handles files being uploaded via POST requests.

@mjforan mjforan requested a review from d-a-v September 17, 2020 20:11
Copy link
Collaborator

@d-a-v d-a-v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, nothing jumps at me after carefully reading.

@earlephilhower earlephilhower self-requested a review October 25, 2020 05:28
Copy link
Collaborator

@earlephilhower earlephilhower left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strangely enough I was unable to reproduce your failure with master using the same PEM as you did, but I do see the issue you found in the existing code. Things have to align poorly in the HTTP stream to hit it, I guess.

Thx for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug in parsing file upload through http form
4 participants