Ticket #14103: django_utils_cache_honor_use_etags_plus_test.diff

django/utils/cache.py

@@ -101 +101 @@
         cache_timeout = settings.CACHE_MIDDLEWARE_SECONDS
     if cache_timeout < 0:
         cache_timeout = 0 # Can't have max-age negative
-    if not response.has_header('ETag'):
+    if settings.USE_ETAGS and not response.has_header('ETag'):
         response['ETag'] = '"%s"' % md5_constructor(response.content).hexdigest()
     if not response.has_header('Last-Modified'):
         response['Last-Modified'] = http_date()

tests/regressiontests/cache/tests.py

@@ -545 +545 @@
         settings.CACHE_MIDDLEWARE_SECONDS = 60
         settings.CACHE_MIDDLEWARE_KEY_PREFIX="test"
         settings.CACHE_BACKEND='locmem:///'
+        settings.USE_ETAGS = True
         settings.USE_I18N = True
         en_message ="Hello world!"
         es_message ="Hola mundo!"
@@ -555 +556 @@
         # Check that we can recover the cache
         self.assertNotEqual(get_cache_data.content, None)
         self.assertEqual(en_message, get_cache_data.content)
+        # Check that we use etags
+        self.assertTrue(get_cache_data.has_header('ETag'))
+        # Check that we can disable etags
+        settings.USE_ETAGS = False
+        request._cache_update_cache = True
+        set_cache(request, 'en', en_message)
+        get_cache_data = FetchFromCacheMiddleware().process_request(request)
+        self.assertFalse(get_cache_data.has_header('ETag'))
         # change the session language and set content
         request = self._get_request_cache()
         set_cache(request, 'es', es_message)