[Django] #25598: support for SCRIPT_NAME for static tag and FileField.url

classic Classic list List threaded Threaded
22 messages Options
12
Reply | Threaded
Open this post in threaded view
|

[Django] #25598: support for SCRIPT_NAME for static tag and FileField.url

Django
#25598: support for SCRIPT_NAME for static tag and FileField.url
--------------------------+------------------------------------------------
     Reporter:            |      Owner:  nobody
  DheerendraRathor        |
         Type:  New       |     Status:  new
  feature                 |
    Component:            |    Version:  master
  Uncategorized           |
     Severity:  Normal    |   Keywords:  script_name, static_url, media_url
 Triage Stage:            |  Has patch:  0
  Unreviewed              |
Easy pickings:  0         |      UI/UX:  0
--------------------------+------------------------------------------------
 By default, `static` tag just appends `STATIC_URL` in the path. When
 running on subpath it results in incorrect static url.

 This problem can be solved with appending `SCRIPT_NAME` in to `SCRIPT_URL`
 in `settings.py`

 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.

--
Ticket URL: <https://code.djangoproject.com/ticket/25598>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/059.8b313d182aa162563c2e83faa03c53c2%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: support for SCRIPT_NAME for static tag and FileField.url

Django
#25598: support for SCRIPT_NAME for static tag and FileField.url
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  Uncategorized        |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:
  static_url, media_url              |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by timgraham):

 * needs_better_patch:   => 0
 * needs_tests:   => 0
 * needs_docs:   => 0


Comment:

 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?

--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:1>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.115df90ca500b01afe6ade412124a620%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: support for SCRIPT_NAME for static tag and FileField.url

Django
In reply to this post by Django
#25598: support for SCRIPT_NAME for static tag and FileField.url
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  Uncategorized        |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:
  static_url, media_url              |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by DheerendraRathor):

 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.

--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:2>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.eb8291d5cb74766bdbe9dd604dcbe825%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: support for SCRIPT_NAME for static tag and FileField.url

Django
In reply to this post by Django
#25598: support for SCRIPT_NAME for static tag and FileField.url
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  Uncategorized        |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:
  static_url, media_url              |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by timgraham):

 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
 [https://docs.djangoproject.com/en/stable/topics/security/#user-uploaded-
 content 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?

--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:3>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.88e8e34ec1716fd56a77a2e9cc774e04%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: support for SCRIPT_NAME for static tag and FileField.url

Django
In reply to this post by Django
#25598: support for SCRIPT_NAME for static tag and FileField.url
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  Uncategorized        |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:
  static_url, media_url              |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by claudep):

 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.

--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:4>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.dd8e05c97fd0e3e732a70ed551b33ca9%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: support for SCRIPT_NAME for static tag and FileField.url

Django
In reply to this post by Django
#25598: support for SCRIPT_NAME for static tag and FileField.url
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:
  static_url, media_url              |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by timgraham):

 * component:  Uncategorized => contrib.staticfiles


Comment:

 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.

--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:5>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.12b43b779a8c30ccc5d88243238aa87b%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL (was: support for SCRIPT_NAME for static tag and FileField.url)

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by timgraham):

 * stage:  Unreviewed => Accepted


--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:6>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.d4fdbb3a30c801df5a6f8465186da89d%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by kottenator):

 * cc: kottenator@… (added)


--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:7>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.455fc3a7d0f6d07e0b09e4b7a78ec82e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by kottenator):

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

--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:8>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.43bb9e6f9c0f9216cf44eddbf6f764c0%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Description changed by kottenator:

Old description:

> By default, `static` tag just appends `STATIC_URL` in the path. When
> running on subpath it results in incorrect static url.
>
> This problem can be solved with appending `SCRIPT_NAME` in to
> `SCRIPT_URL` in `settings.py`
>
> 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.

New description:

 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` in to
 `SCRIPT_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.

--

--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:9>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.03bc46a35f43bc41f6ad2ac43466d706%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Description changed by kottenator:

Old description:

> 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` in to
> `SCRIPT_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.

New description:

 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.

--

--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:10>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.06702918ac9aacefa387324e871b47c0%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by kottenator):

 * has_patch:  0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:11>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.861b2129ef2302dcd3e573a78d987463%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by timgraham):

 * needs_better_patch:  0 => 1


Comment:

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

--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:12>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.cefc04c8fb84365fe1e291636f670641%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  DheerendraRathor     |                    Owner:  nobody
         Type:  New feature          |                   Status:  new
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by kottenator):

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

--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:13>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.4401f8727e53bd3a53df2c5876baf302%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  Dheerendra Rathor    |                    Owner:  jrwdunham
         Type:  New feature          |                   Status:  assigned
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by jrwdunham):

 * owner:  nobody => jrwdunham
 * status:  new => assigned


Comment:

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

--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:14>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.ff752fdad3c4a0c843ebfc909df030b3%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  Dheerendra Rathor    |                    Owner:  Joel
                                     |  Dunham
         Type:  New feature          |                   Status:  assigned
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Oleg Kainov):

 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

--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:15>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.92b191a3e95a891fc5ec8cf610856a5f%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  Dheerendra Rathor    |                    Owner:  Joel
                                     |  Dunham
         Type:  New feature          |                   Status:  assigned
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by felixxm):

 * needs_better_patch:  1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:16>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.ebf254f97bb2093ac899e7c7bee48ccd%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  Dheerendra Rathor    |                    Owner:  Oleg
                                     |  Kainov
         Type:  New feature          |                   Status:  assigned
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by felixxm):

 * owner:  Joel Dunham => Oleg Kainov


--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:17>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.9153a7044dc5ba9b8e2723432e404586%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  Dheerendra Rathor    |                    Owner:  Oleg
                                     |  Kainov
         Type:  New feature          |                   Status:  assigned
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by felixxm):

 * needs_better_patch:  0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:18>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.68479d3a974da4a14c38340581961c5c%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL

Django
In reply to this post by Django
#25598: Add support for SCRIPT_NAME in STATIC_URL and MEDIA_URL
-------------------------------------+-------------------------------------
     Reporter:  Dheerendra Rathor    |                    Owner:  Oleg
                                     |  Kainov
         Type:  New feature          |                   Status:  assigned
    Component:  contrib.staticfiles  |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  script_name,         |             Triage Stage:  Accepted
  static_url, media_url              |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by felixxm):

 * needs_better_patch:  1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/25598#comment:19>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

--
You received this message because you are subscribed to the Google Groups "Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/074.559feaf613e4b91948c5089b589fb843%40djangoproject.com.
12