Opened 8 years ago

Closed 4 years ago

Last modified 3 years ago

#25598 closed New feature (fixed)

Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Reported by: Dheerendra Rathor Owned by: Oleg Kainov
Component: contrib.staticfiles Version: dev
Severity: Normal Keywords: script_name, static_url, media_url
Cc: kottenator@… Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Rostyslav Bryzgunov)

By default, {% static '...' %} tag just appends STATIC_URL in the path. When running on sub-path, using SCRIPT_NAME WSGI param, it results in incorrect static URL - it doesn't prepend SCRIPT_NAME prefix.

This problem can be solved with prepending SCRIPT_NAME to STATIC_URL in settings.py but that doesn't work when SCRIPT_NAME is a dynamic value.

This can be easily added into default Django static tag and django.contrib.staticfiles tag as following:

def render(self, context):
    url = self.url(context)
    # Updating url here with request.META['SCRIPT_NAME'] 
    if self.varname is None:
        return url
    context[self.varname] = url
        return ''

On more research I found that FileSystemStorage and StaticFilesStorage ignores SCRIPT_NAME as well.

We might have to do a lot of changes but I think it's worth the efforts.

Change History (22)

comment:1 by Tim Graham, 8 years ago

This change doesn't seem correct to me (for one, it seems like it could break existing sites). Why not include the appropriate prefix in your STATIC_URL and MEDIA_URL settings?

comment:2 by Dheerendra Rathor, 8 years ago

This is not a patch. This is just an idea I got about the patch for {% static %} only. The patch will (probably) involve FileSystemStorage and StaticFileSystemStorage classes.

The main idea behind this feature was that Django will auto detect script_name header and use that accordingly for creating static and media urls. This will reduce human efforts for setting up sites in future. This patch will also take time to develop so it can be added in Django2.0 timeline.

comment:3 by Tim Graham, 8 years ago

What I meant was that I don't think Django should automatically use SCRIPT_NAME in generating those URLs. If you're running your site on a subpath, then you should set your STATIC_URL to 'http://example.com/subpath/static/' or whatever. However, you might not even be hosting static and uploaded files on the same domain as your site (in fact, for user-uploaded files, you shouldn't do that for security reasons) in which case SCRIPT_URL is irrelevant in constructing the static/media URLs.

How would the change make it easier to setup sites?

comment:4 by Claude Paroz, 8 years ago

I think that the idea basically makes sense. Ideally, a Django instance shouldn't need to know at which subpath it is being deployed, as this can be considered as purely sysadmin stuff. It would be a good separation of concerns. For example, the Web administrator may change the WSGIScriptAlias from /foo to /bar and the application should continue working. Of course, this only applies when *_URL settings are not full URIs.

In practice, it's very likely that many running instances are adapting their *_URL settings to include the base script path, hence the behavior change would be backwards incompatible. The question is whether the change is worth the incompatibility.

comment:5 by Tim Graham, 8 years ago

Component: Uncategorizedcontrib.staticfiles

I see. I guess the idea would be to use get_script_prefix() like reverse() does as I don't think we have access to request everywhere we need it. It seems like some public APIs like get_static_url() and get_media_url() would replace accessing the settings directly whenever building URLs. For backwards compatibility, possibly these functions could try to detect if the setting is already prefixed appropriately.

Removing the prefix from the settings, however, means that the URLs are no longer correct when generated outside of a request/response cycle though (#16734). I'm not sure if it might create any practical problems, but we might think about addressing that issue first.

comment:6 by Tim Graham, 8 years ago

Summary: support for SCRIPT_NAME for static tag and FileField.urlAdd support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
Triage Stage: UnreviewedAccepted

comment:7 by Rostyslav Bryzgunov, 8 years ago

Cc: kottenator@… added

comment:8 by Rostyslav Bryzgunov, 8 years ago

I'm here at DjangoCon US 2016 will try to create a patch for this ticket ;)

Why?

But before I make the patch, here are some reasons to do it.

The first reason is consistency inside Django core:

  • {% url '...' %} template tag does respect SCRIPT_NAME but {% static '...' %} does not
  • reverse(...) function does respect SCRIPT_NAME but static(...) does not

And the second reason is that there is no way to make it work in case when SCRIPT_NAME is a dynamic value - see an example below.

Of course we shouldn't modify STATIC_URL when it's an absolute URL, with domain & protocol. But if it starts with / - it's relative to our Django project and we need to add SCRIPT_NAME prefix.

Real life example

You have Django running via WSGI behind reverse proxy (let's call it back-end server), and another HTTP server on the front (let's call it front-end server). Front-end server URL is http://some.domain.com/sub/path/, back-end server URL is http://1.2.3.4:5678/. You want them both to work. You pass SCRIPT_NAME = '/sub/path/' from front-end server to back-end one. But when you access back-end server directly - there is no SCRIPT_NAME passed to WSGI/Django.

So we cannot hard-code SCRIPT_NAME in Django settings because it's dynamic.

Last edited 8 years ago by Rostyslav Bryzgunov (previous) (diff)

comment:9 by Rostyslav Bryzgunov, 8 years ago

Description: modified (diff)

comment:10 by Rostyslav Bryzgunov, 8 years ago

Description: modified (diff)

comment:11 by Rostyslav Bryzgunov, 8 years ago

Has patch: set
Last edited 8 years ago by Rostyslav Bryzgunov (previous) (diff)

comment:12 by Tim Graham, 8 years ago

Patch needs improvement: set

At least documentation and additional tests look like they are required.

comment:13 by Rostyslav Bryzgunov, 8 years ago

Absolutely agree with your remarks, Tim. I'll add tests. Could you point to docs that need to be updated?

comment:14 by Joel Dunham, 5 years ago

Owner: changed from nobody to Joel Dunham
Status: newassigned

I would like to take this ticket on and have a new PR for it: https://github.com/django/django/pull/10724

comment:15 by Oleg Kainov, 5 years ago

I tried to revive activities around this ticket, so I created (yet another) PR based on previous work https://github.com/django/django/pull/11564

comment:16 by Mariusz Felisiak, 5 years ago

Patch needs improvement: unset

comment:17 by Mariusz Felisiak, 5 years ago

Owner: changed from Joel Dunham to Oleg Kainov

comment:18 by Mariusz Felisiak, 5 years ago

Patch needs improvement: set

comment:19 by Mariusz Felisiak, 5 years ago

Patch needs improvement: unset

comment:20 by Mariusz Felisiak, 4 years ago

Triage Stage: AcceptedReady for checkin

comment:21 by Mariusz Felisiak <felisiak.mariusz@…>, 4 years ago

Resolution: fixed
Status: assignedclosed

In c574bec:

Fixed #25598 -- Added SCRIPT_NAME prefix to STATIC_URL and MEDIA_URL set to relative paths.

Thanks Florian Apolloner for reviews.

Co-authored-by: Joel Dunham <Joel.Dunham@…>

comment:22 by Mariusz Felisiak <felisiak.mariusz@…>, 3 years ago

In 9d05add:

Fixed #32394 -- Changed project template settings to use relative STATIC_URL.

Refs #25598 which added original support for relative STATIC_URL and
MEDIA_URL.

Note: See TracTickets for help on using tickets.
Back to Top