diff --git a/django/contrib/staticfiles/storage.py b/django/contrib/staticfiles/storage.py index 0293363fc4..ac83f522ee 100644 --- a/django/contrib/staticfiles/storage.py +++ b/django/contrib/staticfiles/storage.py @@ -105,13 +105,16 @@ class CachedFilesMixin(object): hashed_name, fragment = name, '' else: clean_name, fragment = urldefrag(name) - cache_key = self.cache_key(name) - hashed_name = self.cache.get(cache_key) - if hashed_name is None: - hashed_name = self.hashed_name(clean_name).replace('\\', '/') - # set the cache if there was a miss - # (e.g. if cache server goes down) - self.cache.set(cache_key, hashed_name) + if urlsplit(clean_name).path.endswith('/'): # don't hash paths + hashed_name = name + else: + cache_key = self.cache_key(name) + hashed_name = self.cache.get(cache_key) + if hashed_name is None: + hashed_name = self.hashed_name(clean_name).replace('\\', '/') + # set the cache if there was a miss + # (e.g. if cache server goes down) + self.cache.set(cache_key, hashed_name) final_url = super(CachedFilesMixin, self).url(hashed_name) diff --git a/tests/regressiontests/staticfiles_tests/tests.py b/tests/regressiontests/staticfiles_tests/tests.py index 680027b63c..9834885fdd 100644 --- a/tests/regressiontests/staticfiles_tests/tests.py +++ b/tests/regressiontests/staticfiles_tests/tests.py @@ -317,6 +317,10 @@ class TestCollectionCachedStorage(BaseCollectionTestCase, "/static/test/file.ea5bccaf16d5.txt") self.assertStaticRenders("cached/styles.css", "/static/cached/styles.93b1147e8552.css") + self.assertStaticRenders("path/", + "/static/path/") + self.assertStaticRenders("path/?query", + "/static/path/?query") def test_template_tag_simple_content(self): relpath = self.cached_file_path("cached/styles.css")