diff options
| author | Claude Paroz <claude@2xlibre.net> | 2013-08-24 18:08:05 +0200 |
|---|---|---|
| committer | Claude Paroz <claude@2xlibre.net> | 2013-08-24 18:10:12 +0200 |
| commit | f4e9804567ce45ef01f6613ce194e96d293ac04b (patch) | |
| tree | f4854553ac93764ad7852eec295750d86bf11090 | |
| parent | f33db5a09acfc3df3085235a5712c46094eb9a0d (diff) | |
Fixed #20961 -- Fixed HttpResponse default empty content
Thanks epandurski at gmail.com for the report.
| -rw-r--r-- | django/http/response.py | 2 | ||||
| -rw-r--r-- | tests/httpwrappers/tests.py | 7 |
2 files changed, 8 insertions, 1 deletions
diff --git a/django/http/response.py b/django/http/response.py index aaebefe022..822589fe81 100644 --- a/django/http/response.py +++ b/django/http/response.py @@ -317,7 +317,7 @@ class HttpResponse(HttpResponseBase): streaming = False - def __init__(self, content='', *args, **kwargs): + def __init__(self, content=b'', *args, **kwargs): super(HttpResponse, self).__init__(*args, **kwargs) # Content is a bytestring. See the `content` property methods. self.content = content diff --git a/tests/httpwrappers/tests.py b/tests/httpwrappers/tests.py index 1679b6e036..4ee342c252 100644 --- a/tests/httpwrappers/tests.py +++ b/tests/httpwrappers/tests.py @@ -385,6 +385,13 @@ class HttpResponseTests(unittest.TestCase): self.assertEqual(r.tell(), 6) self.assertEqual(r.content, b'abcdef') + # with Content-Encoding header + r = HttpResponse() + r['Content-Encoding'] = 'winning' + r.write(b'abc') + r.write(b'def') + self.assertEqual(r.content, b'abcdef') + def test_unsafe_redirect(self): bad_urls = [ 'data:text/html,<script>window.alert("xss")</script>', |
