mirror of https://github.com/python/cpython
Merged revisions 87564 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/branches/py3k ........ r87564 | senthil.kumaran | 2010-12-29 14:25:42 +0800 (Wed, 29 Dec 2010) | 3 lines Fix Issue 10753 - Don't quote ;=, in the PATH_INFO envvar. ........
This commit is contained in:
parent
0df73674e5
commit
559395fde0
|
@ -306,6 +306,10 @@ class UtilityTests(TestCase):
|
|||
self.checkReqURI("http://127.0.0.1/spam", SCRIPT_NAME="/spam")
|
||||
self.checkReqURI("http://127.0.0.1/spammity/spam",
|
||||
SCRIPT_NAME="/spammity", PATH_INFO="/spam")
|
||||
self.checkReqURI("http://127.0.0.1/spammity/spam;ham",
|
||||
SCRIPT_NAME="/spammity", PATH_INFO="/spam;ham")
|
||||
self.checkReqURI("http://127.0.0.1/spammity/spam;cookie=1234,5678",
|
||||
SCRIPT_NAME="/spammity", PATH_INFO="/spam;cookie=1234,5678")
|
||||
self.checkReqURI("http://127.0.0.1/spammity/spam?say=ni",
|
||||
SCRIPT_NAME="/spammity", PATH_INFO="/spam",QUERY_STRING="say=ni")
|
||||
self.checkReqURI("http://127.0.0.1/spammity/spam", 0,
|
||||
|
|
|
@ -64,7 +64,7 @@ def request_uri(environ, include_query=1):
|
|||
"""Return the full request URI, optionally including the query string"""
|
||||
url = application_uri(environ)
|
||||
from urllib import quote
|
||||
path_info = quote(environ.get('PATH_INFO',''))
|
||||
path_info = quote(environ.get('PATH_INFO',''),safe='/;=,')
|
||||
if not environ.get('SCRIPT_NAME'):
|
||||
url += path_info[1:]
|
||||
else:
|
||||
|
|
|
@ -22,6 +22,11 @@ Core and Builtins
|
|||
Library
|
||||
-------
|
||||
|
||||
- Issue 10753 - Characters ';','=' and ',' in the PATH_INFO environment
|
||||
variable won't be quoted when the URI is constructed by the wsgiref.util 's
|
||||
request_uri method. According to RFC 3986, these characters can be a part of
|
||||
params in PATH component of URI and need not be quoted.
|
||||
|
||||
- Issue 10738: Fix webbrowser.Opera.raise_opts
|
||||
|
||||
- Issue 9824: SimpleCookie now encodes , and ; in values to cater to how
|
||||
|
|
Loading…
Reference in New Issue