Make toolbar compatible with FORCE_SCRIPT_NAME

debug_toolbar/panels/request.py

@@ -41,7 +41,7 @@ def generate_stats(self, request, response):
             "view_urlname": "None",
         }
         try:
-            match = resolve(request.path)
+            match = resolve(request.path_info)
             func, args, kwargs = match
             view_info["view_func"] = get_name_from_obj(func)
             view_info["view_args"] = args

debug_toolbar/toolbar.py

@@ -165,7 +165,7 @@ def is_toolbar_request(cls, request):
         # not have resolver_match set.
         try:
             resolver_match = request.resolver_match or resolve(
-                request.path, getattr(request, "urlconf", None)
+                request.path_info, getattr(request, "urlconf", None)
             )
         except Resolver404:
             return False

docs/changes.rst

@@ -4,6 +4,9 @@ Change log
 Pending
 -------
 
+* Fixed internal toolbar requests being instrumented if the Django setting
+  ``FORCE_SCRIPT_NAME`` was set.
+
 4.4.6 (2024-07-10)
 ------------------
 

tests/panels/test_request.py

@@ -1,7 +1,10 @@
 from django.http import QueryDict
+from django.test import RequestFactory
 
 from ..base import BaseTestCase
 
+rf = RequestFactory()
+
 
 class RequestPanelTestCase(BaseTestCase):
     panel_id = "RequestPanel"
@@ -13,21 +16,21 @@ def test_non_ascii_session(self):
         self.assertIn("où", self.panel.content)
 
     def test_object_with_non_ascii_repr_in_request_params(self):
-        self.request.path = "/non_ascii_request/"
-        response = self.panel.process_request(self.request)
-        self.panel.generate_stats(self.request, response)
+        request = rf.get("/non_ascii_request/")
+        response = self.panel.process_request(request)
+        self.panel.generate_stats(request, response)
         self.assertIn("nôt åscíì", self.panel.content)
 
     def test_insert_content(self):
         """
         Test that the panel only inserts content after generate_stats and
         not the process_request.
         """
-        self.request.path = "/non_ascii_request/"
-        response = self.panel.process_request(self.request)
+        request = rf.get("/non_ascii_request/")
+        response = self.panel.process_request(request)
         # ensure the panel does not have content yet.
         self.assertNotIn("nôt åscíì", self.panel.content)
-        self.panel.generate_stats(self.request, response)
+        self.panel.generate_stats(request, response)
         # ensure the panel renders correctly.
         content = self.panel.content
         self.assertIn("nôt åscíì", content)
@@ -99,9 +102,9 @@ def test_list_for_request_in_method_post(self):
         self.assertIn("[{'a': 1}, {'b': 2}]", content)
 
     def test_namespaced_url(self):
-        self.request.path = "/admin/login/"
-        response = self.panel.process_request(self.request)
-        self.panel.generate_stats(self.request, response)
+        request = rf.get("/admin/login/")
+        response = self.panel.process_request(request)
+        self.panel.generate_stats(request, response)
         panel_stats = self.panel.get_stats()
         self.assertEqual(panel_stats["view_urlname"], "admin:login")
 

tests/test_integration.py

@@ -128,10 +128,10 @@ def test_should_render_panels_multiprocess(self):
 
     def _resolve_stats(self, path):
         # takes stats from Request panel
-        self.request.path = path
+        request = rf.get(path)
         panel = self.toolbar.get_panel_by_id("RequestPanel")
-        response = panel.process_request(self.request)
-        panel.generate_stats(self.request, response)
+        response = panel.process_request(request)
+        panel.generate_stats(request, response)
         return panel.get_stats()
 
     def test_url_resolving_positional(self):
@@ -215,34 +215,47 @@ def test_cache_disable_instrumentation(self):
         self.assertEqual(len(response.toolbar.get_panel_by_id("CachePanel").calls), 0)
 
     def test_is_toolbar_request(self):
-        self.request.path = "/__debug__/render_panel/"
-        self.assertTrue(self.toolbar.is_toolbar_request(self.request))
+        request = rf.get("/__debug__/render_panel/")
+        self.assertTrue(self.toolbar.is_toolbar_request(request))
 
-        self.request.path = "/invalid/__debug__/render_panel/"
-        self.assertFalse(self.toolbar.is_toolbar_request(self.request))
+        request = rf.get("/invalid/__debug__/render_panel/")
+        self.assertFalse(self.toolbar.is_toolbar_request(request))
 
-        self.request.path = "/render_panel/"
-        self.assertFalse(self.toolbar.is_toolbar_request(self.request))
+        request = rf.get("/render_panel/")
+        self.assertFalse(self.toolbar.is_toolbar_request(request))
 
     @override_settings(ROOT_URLCONF="tests.urls_invalid")
     def test_is_toolbar_request_without_djdt_urls(self):
         """Test cases when the toolbar urls aren't configured."""
-        self.request.path = "/__debug__/render_panel/"
-        self.assertFalse(self.toolbar.is_toolbar_request(self.request))
+        request = rf.get("/__debug__/render_panel/")
+        self.assertFalse(self.toolbar.is_toolbar_request(request))
 
-        self.request.path = "/render_panel/"
-        self.assertFalse(self.toolbar.is_toolbar_request(self.request))
+        request = rf.get("/render_panel/")
+        self.assertFalse(self.toolbar.is_toolbar_request(request))
 
     @override_settings(ROOT_URLCONF="tests.urls_invalid")
     def test_is_toolbar_request_override_request_urlconf(self):
         """Test cases when the toolbar URL is configured on the request."""
-        self.request.path = "/__debug__/render_panel/"
-        self.assertFalse(self.toolbar.is_toolbar_request(self.request))
+        request = rf.get("/__debug__/render_panel/")
+        self.assertFalse(self.toolbar.is_toolbar_request(request))
 
         # Verify overriding the urlconf on the request is valid.
-        self.request.urlconf = "tests.urls"
-        self.request.path = "/__debug__/render_panel/"
-        self.assertTrue(self.toolbar.is_toolbar_request(self.request))
+        request.urlconf = "tests.urls"
+        self.assertTrue(self.toolbar.is_toolbar_request(request))
+
+    def test_is_toolbar_request_with_script_prefix(self):
+        """
+        Test cases when Django is running under a path prefix, such as via the
+        FORCE_SCRIPT_NAME setting.
+        """
+        request = rf.get("/__debug__/render_panel/", SCRIPT_NAME="/path/")
+        self.assertTrue(self.toolbar.is_toolbar_request(request))
+
+        request = rf.get("/invalid/__debug__/render_panel/", SCRIPT_NAME="/path/")
+        self.assertFalse(self.toolbar.is_toolbar_request(request))
+
+        request = rf.get("/render_panel/", SCRIPT_NAME="/path/")
+        self.assertFalse(self.toolbar.is_toolbar_request(self.request))
 
     def test_data_gone(self):
         response = self.client.get(