[Django] #27325: Offer a solution for static file serving suitable for production use
Django
----------------------------------------+------------------------
Reporter: Tim Graham | Owner: nobody
Type: New feature | Status: new
Component: Core (Other) | Version: master
Severity: Normal | Keywords:
Triage Stage: Accepted | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
----------------------------------------+------------------------
The documentation for the function for serving static files,
`django.contrib.staticfiles.views.serve()`, describes it as "grossly
inefficient and probably insecure. This is only intended for local
development, and should never be used in production." To ease the
deployment of small sites, it would be nice to fix or replace it with
something that could be used in production. [https://whitenoise.evans.io/
WhiteNoise] is an example of static file serving done in Python and is a
possible candidate for integration in Django.
Ideas are discussed in more detail on the
[https://groups.google.com/d/topic/django-
developers/N0KbgDeLuUE/discussion delegating our static file serving]
thread on django-developers.
--
Ticket URL: <https://code.djangoproject.com/ticket/27325>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Django
Reporter: Tim Graham | Owner: Aleksej Manaev
Type: New feature | Status: assigned
Component: Core (Other) | Version: master
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Aleksej Manaev):
* cc: Aleksej Manaev (added)
* owner: nobody => Aleksej Manaev
* status: new => assigned
--
Ticket URL: <https://code.djangoproject.com/ticket/27325#comment:1>
Django
Reporter: Tim Graham | Owner: Aleksej Manaev
Type: New feature | Status: assigned
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Description changed by Mads Jensen:
Old description:
> The documentation for the function for serving static files,
> `django.contrib.staticfiles.views.serve()`, describes it as "grossly
> inefficient and probably insecure. This is only intended for local
> development, and should never be used in production." To ease the
> deployment of small sites, it would be nice to fix or replace it with
> something that could be used in production. [https://whitenoise.evans.io/
> WhiteNoise] is an example of static file serving done in Python and is a
> possible candidate for integration in Django.
>
> Ideas are discussed in more detail on the
> [https://groups.google.com/d/topic/django-
> developers/N0KbgDeLuUE/discussion delegating our static file serving]
> thread on django-developers.
New description:
The documentation for the function for serving static files,
`django.contrib.staticfiles.views.serve()`, describes it as "grossly
inefficient and probably insecure. This is only intended for local
development, and should never be used in production." To ease the
deployment of small sites, it would be nice to fix or replace it with
something that could be used in production. [http://whitenoise.evans.io/
WhiteNoise] is an example of static file serving done in Python and is a
possible candidate for integration in Django.
Ideas are discussed in more detail on the
[https://groups.google.com/d/topic/django-
developers/N0KbgDeLuUE/discussion delegating our static file serving]
thread on django-developers.
--
--
Ticket URL: <https://code.djangoproject.com/ticket/27325#comment:2>
Django
Reporter: Tim Graham | Owner: Aleksej Manaev
Type: New feature | Status: assigned
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Aleksej Manaev):
* has_patch: 0 => 1
Comment:
Pull Request: https://github.com/django/django/pull/8245
--
Ticket URL: <https://code.djangoproject.com/ticket/27325#comment:3>
Django
Reporter: Tim Graham | Owner: Aleksej Manaev
Type: New feature | Status: assigned
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
Changes (by Tim Graham):
* needs_better_patch: 0 => 1
Comment:
I agree with what Florian said on the mailing list, "I think there is no
need to vendor [whitenoise], we can just depend on it. (like we do for
pytz)". The Django-specific parts can be moved in Django. Also, Python 2
compatibility can be removed since the master branch doesn't support it
anymore. Tests and documentation are also required.
--
Ticket URL: <https://code.djangoproject.com/ticket/27325#comment:4>
Django
Reporter: Tim Graham | Owner: Aleksej Manaev
Type: New feature | Status: assigned
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
Comment (by Aleksej Manaev):
If we deprecate django.contrib.staticfiles.handlers.StaticFilesHandler and
replace it by DjangoWhiteNoise in
django.contrib.staticfiles.testing.StaticLiveServerTestCase it would not
be possible to run tests without using collectstaic first. Is this an
important behavior?
tests/staticfiles_tests/test_liverserver.StaticLiveServerView.test_collectstatic_emulation
only passes with DEBUG=True.
WHITENOISE_USE_FINDERS=True causes:
"django.core.exceptions.ImproperlyConfigured: WHITENOISE_USE_FINDERS can
only be enabled in development when WHITENOISE_AUTOREFRESH is also
enabled."
--
Ticket URL: <https://code.djangoproject.com/ticket/27325#comment:5>
Django
Reporter: Tim Graham | Owner: Aleksej Manaev
Type: New feature | Status: assigned
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Aleksej Manaev):
* needs_better_patch: 1 => 0
--
Ticket URL: <https://code.djangoproject.com/ticket/27325#comment:6>
Django
Reporter: Tim Graham | Owner: Aleksej Manaev
Type: New feature | Status: assigned
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 1
Easy pickings: 0 | UI/UX: 0
Changes (by Tim Graham):
* needs_better_patch: 0 => 1
--
Ticket URL: <https://code.djangoproject.com/ticket/27325#comment:7>
Django
Reporter: Tim Graham | Owner: (none)
Type: New feature | Status: new
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Carlton Gibson):
* owner: Aleksej Manaev => (none)
* needs_better_patch: 1 => 0
* has_patch: 1 => 0
* status: assigned => new
--
Ticket URL: <https://code.djangoproject.com/ticket/27325#comment:8>
Django
Reporter: Tim Graham | Owner: (none)
Severity: Normal | Resolution:
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by Ülgen Sarıkavak):
* cc: Ülgen Sarıkavak (added)
--
Ticket URL: <https://code.djangoproject.com/ticket/27325#comment:9>