diff options
| author | Tim Graham <timograham@gmail.com> | 2015-05-27 09:19:19 -0400 |
|---|---|---|
| committer | Tim Graham <timograham@gmail.com> | 2015-05-27 09:29:45 -0400 |
| commit | b16f84f15b1344d2a3df8149565cfc8de803eb77 (patch) | |
| tree | 82d61d75b5730cd70fb4867a36391c23398bf71e | |
| parent | 1e3741b8a3d35e46b50d6490e0d7f5048de0dc86 (diff) | |
[1.8.x] Refs #24836 -- Reverted "Simplified the lazy CSRF token implementation in csrf context processor."
This reverts commit 8099d33b6553c9ee7de779ae9d191a1bf22adbda as it caused
a regression that cannot be solved without changing force_text() which has
a small risk of introducing regressions. This change will remain in master
along with an update to force_text().
| -rw-r--r-- | django/template/context_processors.py | 6 | ||||
| -rw-r--r-- | docs/releases/1.8.3.txt | 3 | ||||
| -rw-r--r-- | tests/csrf_tests/test_context_processor.py | 15 |
3 files changed, 22 insertions, 2 deletions
diff --git a/django/template/context_processors.py b/django/template/context_processors.py index a81fe71829..dcd737f4ae 100644 --- a/django/template/context_processors.py +++ b/django/template/context_processors.py @@ -11,8 +11,9 @@ from __future__ import unicode_literals from django.conf import settings from django.middleware.csrf import get_token +from django.utils import six from django.utils.encoding import smart_text -from django.utils.functional import SimpleLazyObject, lazy +from django.utils.functional import lazy def csrf(request): @@ -29,8 +30,9 @@ def csrf(request): return 'NOTPROVIDED' else: return smart_text(token) + _get_val = lazy(_get_val, six.text_type) - return {'csrf_token': SimpleLazyObject(_get_val)} + return {'csrf_token': _get_val()} def debug(request): diff --git a/docs/releases/1.8.3.txt b/docs/releases/1.8.3.txt index 1155ac1cb6..d965630c77 100644 --- a/docs/releases/1.8.3.txt +++ b/docs/releases/1.8.3.txt @@ -19,3 +19,6 @@ Bugfixes ``Count()`` (:ticket:`24835`). * Corrected ``HStoreField.has_changed()`` (:ticket:`24844`). + +* Reverted an optimization to the CSRF template context processor which caused + a regression (:ticket:`24836`). diff --git a/tests/csrf_tests/test_context_processor.py b/tests/csrf_tests/test_context_processor.py new file mode 100644 index 0000000000..270b3e4771 --- /dev/null +++ b/tests/csrf_tests/test_context_processor.py @@ -0,0 +1,15 @@ +import json + +from django.http import HttpRequest +from django.template.context_processors import csrf +from django.test import SimpleTestCase +from django.utils.encoding import force_text + + +class TestContextProcessor(SimpleTestCase): + + def test_force_text_on_token(self): + request = HttpRequest() + request.META['CSRF_COOKIE'] = 'test-token' + token = csrf(request).get('csrf_token') + self.assertEqual(json.dumps(force_text(token)), '"test-token"') |
