Opened 3 years ago
Last modified 3 years ago
#33180 new Cleanup/optimization
Debug 500 HTML broken with strict Content-Security-Policy (CSP) — at Version 1
Reported by: | Adam Johnson | Owned by: | |
---|---|---|---|
Component: | Error reporting | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
When using a strict CSP header, the debug view is broken as the inline CSS and JS get blocked by the browser. The page is somewhat usable as all the text is all visible, but it isn't particularly friendly or necessarily obvious why it's broken.
For example:
I thought of a couple of potential fixes:
- Make the debug http response a special class that deletes any CSP header added to it. This would work for most cases but not if the CSP header is added outside of Django (WSGI middleware, a server wrapping runserver).
- Move the CSS and JS to separate URL's so they're compatible with CSP. They could be served by a special route/path in
runserver
.
I am personally leaning towards #2 - even though it's more complex, it is more general.
There's also some opportunity to modernize the CSS and JS in the debug page a bit, such as rewriting use of window.onload and onclick handlers.
To reproduce, you can save the below as app.py
and run python app.py runserver
:
import sys from django.conf import settings from django.urls import path settings.configure( DEBUG=True, ROOT_URLCONF=__name__, SECRET_KEY="django-insecure-r42jn$xf4g+=w@=l#m6ghqo0!$icww-h4+$5gojq(1ld$x%!6f", MIDDLEWARE=[f"{__name__}.CSPMiddleware"], ) class CSPMiddleware: def __init__(self, get_response): self.get_response = get_response def __call__(self, request): response = self.get_response(request) response[ "Content-Security-Policy" ] = "object-src 'none'; base-uri 'none'; default-src 'self';" return response def index(request): 1 / 0 urlpatterns = [path("", index)] if __name__ == "__main__": from django.core.management import execute_from_command_line execute_from_command_line(sys.argv)