close
The Wayback Machine - https://web.archive.org/web/20211022003809/https://github.com/python/cpython/pull/990
Skip to content
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

bpo-29979: rewrite cgi.parse_multipart() #990

Closed
wants to merge 33 commits into from
Closed

bpo-29979: rewrite cgi.parse_multipart() #990

wants to merge 33 commits into from

Conversation

@PierreQuentel
Copy link
Contributor

@PierreQuentel PierreQuentel commented Apr 4, 2017

The Pull Request simplifies the code of cgi.parse_multipart() by reusing the FieldStorage class, and makes it consistent with FieldStorage results : the values for non-file fields becomes a list of strings, instead of a list of bytes.

Pierre Quentel and others added 30 commits Feb 25, 2017
… check that it's a valid HTTP date with timezone set to UTC.
…format ; ignore ill-formed values in If-Modified-Since header.
…several methods ; add a test for Last-Modified header.
@mention-bot
Copy link

@mention-bot mention-bot commented Apr 4, 2017

@PierreQuentel, thanks for your PR! By analyzing the history of the files in this pull request, we identified @rhettinger, @orsenthil and @collinw to be potential reviewers.

@PierreQuentel
Copy link
Contributor Author

@PierreQuentel PierreQuentel commented Apr 4, 2017

Sorry, I'm not using the correct version of the Github master repo. I close this PR and will open another one soon.

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