Opened 29 hours ago
Last modified 28 hours ago
#36653 new Bug
FORCE_SCRIPT_NAME is not respected for static URLs — at Version 3
Reported by: | Brian Helba | Owned by: | |
---|---|---|---|
Component: | contrib.staticfiles | Version: | 5.2 |
Severity: | Normal | Keywords: | |
Cc: | Brian Helba | Triage Stage: | Unreviewed |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
The documentation for the STATIC_URL
setting states:
If
STATIC_URL
is a relative path, then it will be prefixed by the server-provided value ofSCRIPT_NAME
(or / if not set). This makes it easier to serve a Django application in a subpath without adding an extra configuration to the settings.
However, when using FORCE_SCRIPT_NAME
, the value of STATIC_URL
when serving requests is never updated appropriately. This breaks URL construction via django.templatetags.static.static
(used in templates as {% static ... %}
).
For example, this causes the Django Admin pages to break when using FORCE_SCRIPT_NAME
to serve Django under a subpath. Generally, using FORCE_SCRIPT_NAME
causes Django to behave incorrectly: view URLs are constructed to respect it, static URLs are constructed to not respect it.
I believe that this bug likely also affects static URLs when using the SCRIPT_NAME
WSGI environment variable too, but I haven't verified that yet.
There is definitely some history here, but I believe previous bug reports have struggled to articulate this problem. I'm including all of the below detail in the hope that this issue will be understood well enough to acknowledge it as a legitimate bug.
Long ago, the following reports were made, which I believe are not relevant to this problem (but I'm summarizing them because they've been claimed to be duplicates of this problem):
- #7930 only discussed view URLs (and using
reverse()
); this now works as expected, but static URLs are still broken - #30634 is probably irrelevant; it claimed vague problems with
SCRIPT_NAME
and runserver; the problem here is general to all servers, including both runserver and WSGI - #31724 is probably a true duplicate of #7930
More recently, we've seen:
- #34892 is probably the same as this problem, but the reporter struggled to articulate the behavior and I believe it was mistakenly closed as a duplicate of the aforementioned old issues
- #35985 claimed that this is problem is limited to using threading within management commands, then got derailed by the niche use case and suggestions around the low-level
set_script_prefix
API; it does contain the very useful suggestion to invokedjango.setup()
in each thread, which I don't believe was adequately explored
I believe that I understand the exact cause of the bug. Note, my use of some lifecycle events and specific thread names may be limited to the runserver case, but the exact same behaviors manifest with WSGI (and I believe that equivalent things are occurring with a multiprocess lifecycle).
django.setup()
is called very early by runserverdjango.setup()
callsset_script_prefix
set_script_prefix
correctly setsdjango.urls.base._prefixes.value
, but only for the current thread (since_prefixes
is a thread-local object)- a new thread,
django-main-thread
, is spawned; again, I believe (and can locate if necessary) that an equivalent event also happens in a WSGI lifecycle check_url_settings
runs and accessessettings.STATIC_URL
; importantly, this is the first time in the startup lifecycle thatsettings.STATIC_URL
has ever been accesseddjango.conf.__getattr__
has special logic forSTATIC_URL
, so it calls the staticmethodLazySettings._add_script_prefix
LazySettings._add_script_prefix
callsget_script_prefix
get_script_prefix
looks atdjango.urls.base._prefixes.value
, but it's running in a new thread (step 4), so it doesn't contain theFORCE_SCRIPT_NAME
value (step 3) and returns"/"
django.conf.__getattr__
(step 6) permanently caches the incorrect value ofsettings.STATIC_URL
inLazySettings.__dict__
(which is not thread-local); all future requests forsettings.STATIC_URL
will receive the incorrect value ("/"
, instead ofsettings.FORCE_SCRIPT_NAME
)- The first HTTP request comes in
WSGIHandler.__call__
callsget_script_name
get_script_name
correctly returnssettings.FORCE_SCRIPT_NAME
WSGIHandler.__call__
callsset_script_prefix
with the correct argument (the valuesettings.FORCE_SCRIPT_NAME
)set_script_prefix
(just as in step 3) finally setsdjango.urls.base._prefixes.value
(which, remember, is a thread-local variable, but will persist for at least the remainder of this HTTP request) with the correct value- While rendering the HTTP response, a template or some code calls the function
django.templatetags.static
; assume that the app"django.contrib.staticfiles"
is installed and configured to use some subclass ofStaticFilesStorage
(which is Django's typical configuration) django.contrib.staticfiles.storage.staticfiles_storage.url()
is calledstaticfiles_storage
(an instance ofStaticFilesStorage
) is lazily constructedStaticFilesStorage.__init__
is called; assume it has no arguments fromsettings.STORAGES["staticfiles"]["OPTIONS"]
(this is Django's default)StaticFilesStorage.__init__
defaults to set itsself._base_url
tosettings.STATIC_URL
, butsettings.STATIC_URL
returns an incorrect value (step 9)- Continuing the call in step 16,
FilesystemStorage.url
forms the actual URL fromself.base_url
self.base_url
, a cached property, relies on the incorrectself._base_url
- A static file URL is returned with the incorrect base URL, not respecting
settings.FORCE_SCRIPT_NAME
- Subsequent HTTP requests skip steps 17-19, but otherwise reply the work from step 10 onwards (and also result in incorrect static URLs)
In summary, the problem is that although there's code to attempt to set (via set_script_prefix
) the correct script name both on Django's startup and again on every individual HTTP request, settings.STATIC_URL
slips between a lifecycle "crack" and ends up with the wrong value (which doesn't incorporate the script name, contrary to its documentation), which persists over the entire request-response lifecycle.
Change History (3)
comment:1 by , 29 hours ago
Description: | modified (diff) |
---|
comment:2 by , 29 hours ago
Description: | modified (diff) |
---|
comment:3 by , 29 hours ago
Description: | modified (diff) |
---|