Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Fixed the breakage in [6164] in a different, better way: HttpResponse…

… now implements __contains__ along with __get/set/delitem__, as it should. This is a bit more robust, and should prevent similar breakage from user code.

git-svn-id: http://code.djangoproject.com/svn/django/trunk@6221 bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
commit 4100eab823202a26a7b8227d9b2d1a8c7811c1d7 1 parent 5a5d8a9
@jacobian jacobian authored
Showing with 3 additions and 1 deletion.
  1. +1 −1  django/core/handlers/base.py
  2. +2 −0  django/http/__init__.py
View
2  django/core/handlers/base.py
@@ -142,7 +142,7 @@ def fix_location_header(request, response):
Code constructing response objects is free to insert relative paths and
this function converts them to absolute paths.
"""
- if 'location' in response.headers and http.get_host(request):
+ if 'Location' in response and http.get_host(request):
response['Location'] = request.build_absolute_uri(response['Location'])
return response
View
2  django/http/__init__.py
@@ -273,6 +273,8 @@ def has_header(self, header):
"Case-insensitive check for a header"
return self.headers.has_key(header.lower())
+ __contains__ = has_header
+
def set_cookie(self, key, value='', max_age=None, expires=None, path='/', domain=None, secure=None):
self.cookies[key] = value
for var in ('max_age', 'path', 'domain', 'secure', 'expires'):

0 comments on commit 4100eab

Please sign in to comment.
Something went wrong with that request. Please try again.