"View Site" in admin shows wrong URL if site is not hosted at root

[Dheerendra Rathor]

In django-admin VIEW_SITE should redirect to base of hosted site. If site is not hosted at the root but at some other location, then still VIEW_SITE redirects to absolute root of the site.

Example:
Suppose site is hosted at http://example.com/mysite/ and I'm using SCRIPT_NAME HTTP header in my proxy server. {% url 'some_url' %} results into /mysite/some/url/ which is correct. But site_url in django admin is hardcoded to /.

Here is the relevant code snippet from django.contrib.admin.sites.AdminSite

# URL for the "View site" link at the top of each admin page.
site_url = '/'

[Tim Graham]

Maybe it would work to use get_script_prefix() for the site url if self.site_url == '/' in AdminSite.each_context().

[Dheerendra Rathor]

Replying to timgraham:

Maybe it would work to use get_script_prefix() for the site url if self.site_url == '/' in AdminSite.each_context().

I replaced 'site_url': self.site_url, in my django.contrib.admin.sites.AdminSite#each_context to 'site_url': get_script_prefix(), and then I got expected result.

IMHO we don't need self.site_url == '/' check since site_url is used only once and get_script_prefix() returns / for sites hosted at root.

[Tim Graham]

The if statement is needed because if someone customizes site_url, then we don't want to ignore the value that they set.

[Tim Graham]

​PR

[Tim Graham <timograham@…>]

In 59e85f0:

Fixed #25519 -- Made the admin "View site" link point to sites running on a subpath.

Used request.METASCRIPT_NAME as the site_url if it hasn't been
customized from the default value of '/'.