Merge pull request #1024 from lsst-sqre/tickets/DM-44269

DM-44289: Remove /auth/analyze routes
lsst-sqre · May 9, 2024 · a8ada27 · a8ada27
2 parents 23c5e16 + d08ee01
commit a8ada27
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 300 deletions.
diff --git a/changelog.d/20240509_084145_rra_DM_44269.md b/changelog.d/20240509_084145_rra_DM_44269.md
@@ -0,0 +1,3 @@
+### Backwards-incompatible changes
+
+- Remove the `/auth/analyze` route. This was an old way for a user to see information about their token that has been deprecated for many releases. The output used the old JWT token claim format and was missing a great deal of useful information. `/auth/api/v1/user-info` and `/auth/api/v1/token-info` should be used instead.
diff --git a/src/gafaelfawr/handlers/analyze.py b/src/gafaelfawr/handlers/analyze.py
diff --git a/src/gafaelfawr/main.py b/src/gafaelfawr/main.py
@@ -28,7 +28,7 @@
 from .dependencies.config import config_dependency
 from .dependencies.context import context_dependency
 from .exceptions import DatabaseSchemaError
-from .handlers import analyze, api, auth, cadc, internal, login, logout, oidc
+from .handlers import api, auth, cadc, internal, login, logout, oidc
 from .middleware.state import StateMiddleware
 from .models.state import State
 
@@ -129,7 +129,6 @@ async def lifespan(app: FastAPI) -> AsyncIterator[None]:
     )
 
     # Add all of the routes.
-    app.include_router(analyze.router)
     app.include_router(
         api.router,
         responses={

diff --git a/tests/handlers/analyze_test.py b/tests/handlers/analyze_test.py