languages/cpython
languages/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython synced 2024-09-21 05:21:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

Patch by Jim Fulton, who writes:

Browse source 
"""
The FieldStorage constructor calls the read_multi method.  The read_multi
method creates new FieldStorage objects, re-invoking the constructor
(on the new objects).  The problem is that the 'environ', 'keep_blank_values',
and 'strict_parsing' arguments originally passed to the constructor are not
propigated to the new object constructors.  This causes os.environ to be used,
leading to a miss-handling of the parts.

I fixed this by passing these arguments to read_multi and then on to the
constructor.  See the context diff below.
"""
This commit is contained in:
Guido van Rossum 1998-10-20 14:43:02 +00:00
parent fbddddee49
commit f5745008d2
1 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
Lib/cgi.py
View file

@ -852,7 +852,7 @@ def __init__(self, fp=None, headers=None, outerboundary="",
if ctype == 'application/x-www-form-urlencoded':
self.read_urlencoded()
elif ctype[:10] == 'multipart/':
self.read_multi()
self.read_multi(environ, keep_blank_values, strict_parsing)
else:
self.read_single()
@ -919,14 +919,16 @@ def read_urlencoded(self):
self.list.append(MiniFieldStorage(key, value))
self.skip_lines()
def read_multi(self):
def read_multi(self, environ, keep_blank_values, strict_parsing):
"""Internal: read a part that is itself multipart."""
self.list = []
part = self.__class__(self.fp, {}, self.innerboundary)
part = self.__class__(self.fp, {}, self.innerboundary,
environ, keep_blank_values, strict_parsing)
# Throw first part away
while not part.done:
headers = rfc822.Message(self.fp)
part = self.__class__(self.fp, headers, self.innerboundary)
part = self.__class__(self.fp, headers, self.innerboundary,
environ, keep_blank_values, strict_parsing)
self.list.append(part)
self.skip_lines()