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:50 +0200 |
| commit | 2c08d474a83b438cf02595c6854f2f79e708b852 (patch) | |
| tree | d0fc36d074db6e70076277b1e033a53fb9569810 | |
| parent | 2a166623a63b27a2b6c5c9d0f46bd726ee4e82fb (diff) | |
[1.6.x] Fixed #20961 -- Fixed HttpResponse default empty content
Thanks epandurski at gmail.com for the report.
Backport of f4e980456 from master.
| -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 8d8db2c8b4..f522c0684b 100644 --- a/django/http/response.py +++ b/django/http/response.py @@ -323,7 +323,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 c4c7996aa5..8e0c415623 100644 --- a/tests/httpwrappers/tests.py +++ b/tests/httpwrappers/tests.py @@ -398,6 +398,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>', |
