Removing lazy import

Author: davidpcls

bluesky_httpserver/app.py

@@ -16,6 +16,7 @@
 from fastapi.openapi.utils import get_openapi
 
 from .authentication import ExternalAuthenticator, InternalAuthenticator
+from .authenticators import ProxiedOIDCAuthenticator
 from .console_output import CollectPublishedConsoleOutput, ConsoleOutputStream, SystemInfoStream
 from .core import PatchedStreamingResponse
 from .database.core import purge_expired
@@ -181,11 +182,6 @@ def build_app(authentication=None, api_access=None, resource_access=None, server
         # Authenticators provide Router(s) for their particular flow.
         # Collect them in the authentication_router.
 
-        # Lazy import: avoid importing authenticators at module import time so
-        # that lightweight test doubles that pass fake authenticators don't
-        # need heavy optional dependencies (e.g. python-jose, httpx).
-        from .authenticators import ProxiedOIDCAuthenticator
-
         for spec in authentication["providers"]:
             provider = spec["provider"]
             authenticator = spec["authenticator"]

bluesky_httpserver/database/orm.py

@@ -182,7 +182,7 @@ class Session(Timestamped, Base):
     # Free-form state supplied by the authenticator via UserSessionState.state
     # (e.g. an upstream OIDC access_token/refresh_token for OBO exchange).
     # Persisted so it survives across refresh_session calls; the values are
-    # available to downstream services through Tiled access tokens.
+    # available to downstream services through access tokens.
     state = Column(JSON, nullable=False, default=dict, server_default="{}")
 
     principal = relationship("Principal", back_populates="sessions")