Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(81785)

#27777: cgi.FieldStorage can't parse simple body with Content-Length and no Content-Disposition

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 11 months ago by rr-
Modified:
2 years, 11 months ago
Reviewers:
berker.peksag
CC:
Jack.Jansen_cwi.nl, srittau, cito_online.de, ned.deily, pierre.quentel_gmail.com, X-Istence, berkerpeksag, Decorater, rr-_sakuya.pl, remi.lapeyre, chris_eykamp.com, xistence_0x58.com, aronp_guaranteedplus.com, elgow_uw.edu
Visibility:
Public.

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats Patch
Lib/cgi.py View 2 chunks +1 line, -5 lines 0 comments Download
Lib/test/test_cgi.py View 1 chunk +14 lines, -1 line 1 comment Download

Messages

Total messages: 1
berkerpeksag
2 years, 11 months ago #1
http://bugs.python.org/review/27777/diff/19190/Lib/test/test_cgi.py
File Lib/test/test_cgi.py (right):

http://bugs.python.org/review/27777/diff/19190/Lib/test/test_cgi.py#newcode358
Lib/test/test_cgi.py:358: self.assertEqual(fs.list, None)
We prefer to use modern assertions in new code so could you change this to use
assertIsNone instead?
Sign in to reply to this message.

RSS Feeds Recent Issues | This issue
This is Rietveld 894c83f36cb7+