summaryrefslogtreecommitdiff
path: root/django
diff options
context:
space:
mode:
authorAymeric Augustin <aymeric.augustin@m4x.org>2013-09-07 10:25:43 -0500
committerAymeric Augustin <aymeric.augustin@m4x.org>2013-09-07 10:25:43 -0500
commit8aaca651cf5732bbf395d24a7d9f2edfab00250c (patch)
tree314d71ef8fc02da645f90fb77ae8d8fbf5de05a4 /django
parentae7f9afaf6592fa477792647bd8cee0a8560b4eb (diff)
Fixed #20557 -- Properly decoded non-ASCII cookies on Python 3.
Thanks mitsuhiko for the report. Non-ASCII values are supported. Non-ASCII keys still aren't, because the current parser mangles them. That's another bug.
Diffstat (limited to 'django')
-rw-r--r--django/core/handlers/wsgi.py18
1 files changed, 16 insertions, 2 deletions
diff --git a/django/core/handlers/wsgi.py b/django/core/handlers/wsgi.py
index c3d359caf7..7f6d37b3c1 100644
--- a/django/core/handlers/wsgi.py
+++ b/django/core/handlers/wsgi.py
@@ -136,7 +136,8 @@ class WSGIRequest(http.HttpRequest):
def _get_get(self):
if not hasattr(self, '_get'):
# The WSGI spec says 'QUERY_STRING' may be absent.
- self._get = http.QueryDict(self.environ.get('QUERY_STRING', ''), encoding=self._encoding)
+ raw_query_string = get_bytes_from_wsgi(self.environ, 'QUERY_STRING', '')
+ self._get = http.QueryDict(raw_query_string, encoding=self._encoding)
return self._get
def _set_get(self, get):
@@ -152,7 +153,8 @@ class WSGIRequest(http.HttpRequest):
def _get_cookies(self):
if not hasattr(self, '_cookies'):
- self._cookies = http.parse_cookie(self.environ.get('HTTP_COOKIE', ''))
+ raw_cookie = get_str_from_wsgi(self.environ, 'HTTP_COOKIE', '')
+ self._cookies = http.parse_cookie(raw_cookie)
return self._cookies
def _set_cookies(self, cookies):
@@ -265,3 +267,15 @@ def get_bytes_from_wsgi(environ, key, default):
# decoded with ISO-8859-1. This is wrong for Django websites where UTF-8
# is the default. Re-encode to recover the original bytestring.
return value if six.PY2 else value.encode(ISO_8859_1)
+
+
+def get_str_from_wsgi(environ, key, default):
+ """
+ Get a value from the WSGI environ dictionary as bytes.
+
+ key and default should be str objects. Under Python 2 they may also be
+ unicode objects provided they only contain ASCII characters.
+ """
+ value = environ.get(str(key), str(default))
+ # Same comment as above
+ return value if six.PY2 else value.encode(ISO_8859_1).decode(UTF_8)