bpo-29979: rewrite cgi.parse_multipart() by PierreQuentel · Pull Request #990 · python/cpython
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 34.4k
Closed
bpo-29979: rewrite cgi.parse_multipart()#990
PierreQuentel wants to merge 33 commits intopython:masterfrom
PierreQuentel:parse_multipart
bpo-29979: rewrite cgi.parse_multipart()#990
PierreQuentel wants to merge 33 commits intopython:masterfrom
PierreQuentel:parse_multipart
Conversation
Copy link Copy Markdown
Contributor
PierreQuentel
commented
Apr 4, 2017
PierreQuentel
commented
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
February 25, 2017 21:44…Match is present. Add tests.
… check that it's a valid HTTP date with timezone set to UTC.
…osecond=0) instead of int()
…format ; ignore ill-formed values in If-Modified-Since header.
…several methods ; add a test for Last-Modified header.
Pierre Quentel added 3 commits
April 1, 2017 22:28
the-knights-who-say-ni
added
the
CLA signed
label
Copy link Copy Markdown
mention-bot
commented
Apr 4, 2017
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.
Copy link Copy Markdown
Contributor Author
PierreQuentel
commented
Apr 4, 2017
PierreQuentel commented
Apr 4, 2017Sorry, I'm not using the correct version of the Github master repo. I close this PR and will open another one soon.
PierreQuentel
closed this
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants