summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Graham <timograham@gmail.com>2015-05-27 09:19:19 -0400
committerTim Graham <timograham@gmail.com>2015-05-27 09:29:45 -0400
commitb16f84f15b1344d2a3df8149565cfc8de803eb77 (patch)
tree82d61d75b5730cd70fb4867a36391c23398bf71e
parent1e3741b8a3d35e46b50d6490e0d7f5048de0dc86 (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.py6
-rw-r--r--docs/releases/1.8.3.txt3
-rw-r--r--tests/csrf_tests/test_context_processor.py15
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"')