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: bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
1 parent 5a5d8a9 commit 4100eab823202a26a7b8227d9b2d1a8c7811c1d7 @jacobian jacobian committed Sep 14, 2007
Showing with 3 additions and 1 deletion.
  1. +1 −1 django/core/handlers/
  2. +2 −0 django/http/
@@ -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
@@ -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.