diff options
| author | Aymeric Augustin <aymeric.augustin@m4x.org> | 2016-02-20 20:54:18 +0100 |
|---|---|---|
| committer | Aymeric Augustin <aymeric.augustin@m4x.org> | 2016-02-23 19:34:21 +0100 |
| commit | 706b33fef80b8b0901fdd2e954e5e5ea189a528a (patch) | |
| tree | 291c65ceca049cb3d648a865da2158b2f0fbc289 /tests/staticfiles_tests/project/documents/cached | |
| parent | c62807968d7930bfd34afc2036c67921b943592f (diff) | |
Fixed #26249 -- Fixed collectstatic crash for files in STATIC_ROOT referenced by absolute URL.
collectstatic crashed when:
* a hashing static file storage backend was used
* a static file referenced another static file located directly in
STATIC_ROOT (not a subdirectory) with an absolute URL (which must
start with STATIC_URL, which cannot be empty)
It seems to me that the current code reimplements relative path joining
and doesn't handle edge cases correctly. I suspect it assumes that
STATIC_URL is of the form r'/[^/]+/'.
Throwing out that code in favor of the posixpath module makes the logic
easier to follow. Handling absolute paths correctly also becomes easier.
Diffstat (limited to 'tests/staticfiles_tests/project/documents/cached')
| -rw-r--r-- | tests/staticfiles_tests/project/documents/cached/absolute.css | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/staticfiles_tests/project/documents/cached/absolute.css b/tests/staticfiles_tests/project/documents/cached/absolute.css index 03f2ae791a..6a2040b413 100644 --- a/tests/staticfiles_tests/project/documents/cached/absolute.css +++ b/tests/staticfiles_tests/project/documents/cached/absolute.css @@ -1,4 +1,5 @@ @import url("/static/cached/styles.css"); +@import url("/static/styles_root.css"); body { background: #d3d6d8 url(/static/cached/img/relative.png); } |