Removed script URL types in favor of str

Author: mongkok

debug_toolbar/panels/__init__.py

@@ -1,7 +1,6 @@
 import typing as t
 
 from fastapi import Request, Response
-from starlette.datastructures import URL
 from starlette.middleware.base import RequestResponseEndpoint
 
 from debug_toolbar.types import ServerTiming, Stats
@@ -67,11 +66,11 @@ def content(self) -> str:
     def render(self, **context: t.Any) -> str:
         return self.toolbar.render(self.template, **context)
 
-    def url_for(self, name: str, **path_params: t.Any) -> URL:
-        return self.toolbar.request.url_for(name, **path_params)
+    def url_for(self, name: str, **path_params: t.Any) -> str:
+        return str(self.toolbar.request.url_for(name, **path_params))
 
     @property
-    def scripts(self) -> t.List[URL]:
+    def scripts(self) -> t.List[str]:
         return []
 
     async def process_request(self, request: Request) -> Response:

debug_toolbar/panels/timer.py

@@ -2,7 +2,6 @@
 from time import perf_counter
 
 from fastapi import Request, Response
-from starlette.datastructures import URL
 
 from debug_toolbar.panels import Panel
 from debug_toolbar.types import ServerTiming, Stats
@@ -46,7 +45,7 @@ def content(self) -> str:
         return self.render(rows=rows)
 
     @property
-    def scripts(self) -> t.List[URL]:
+    def scripts(self) -> t.List[str]:
         scripts = super().scripts
         scripts.append(self.url_for("debug_toolbar.static", path="js/timer.js"))
         return scripts

debug_toolbar/panels/versions.py

@@ -1,7 +1,6 @@
 import typing as t
 
 from fastapi import Request, Response, __version__
-from starlette.datastructures import URL
 
 from debug_toolbar.panels import Panel
 from debug_toolbar.types import Stats
@@ -16,7 +15,7 @@ def nav_subtitle(self) -> str:
         return f"FastAPI {__version__}"
 
     @property
-    def scripts(self) -> t.List[URL]:
+    def scripts(self) -> t.List[str]:
         scripts = super().scripts
         scripts.append(self.url_for("debug_toolbar.static", path="js/versions.js"))
         return scripts