From 0fcfc745ff5eeb530fd5dcd79643e57fb5365d89 Mon Sep 17 00:00:00 2001 From: Fusion Date: Tue, 19 May 2026 10:56:11 +0200 Subject: [PATCH] fix: add alias=session to Cookie dependencies FastAPI uses parameter name as cookie name by default. get_current_user_id was looking for 'session_cookie' but we set the cookie as 'session'. Add alias='session' to match. --- apps/api/src/api/auth.py | 2 -- apps/api/src/auth/dependencies.py | 10 ++-------- 2 files changed, 2 insertions(+), 10 deletions(-) diff --git a/apps/api/src/api/auth.py b/apps/api/src/api/auth.py index 71d8d49..a6e157e 100644 --- a/apps/api/src/api/auth.py +++ b/apps/api/src/api/auth.py @@ -151,9 +151,7 @@ async def me( session_cookie: str | None = Cookie(default=None, alias="session"), session: AsyncSession = Depends(get_db_session), ) -> dict[str, Any]: - logger.info("/auth/me called, session_cookie present: %s", bool(session_cookie)) if not session_cookie: - logger.warning("/auth/me: no session cookie") raise HTTPException(status_code=status.HTTP_401_UNAUTHORIZED, detail="missing session") settings = Settings() diff --git a/apps/api/src/auth/dependencies.py b/apps/api/src/auth/dependencies.py index c322daa..f177b35 100644 --- a/apps/api/src/auth/dependencies.py +++ b/apps/api/src/auth/dependencies.py @@ -16,16 +16,10 @@ async def get_db_session(): async def get_current_user_id( - session_cookie: Annotated[str | None, Cookie()] = None, + session_cookie: Annotated[str | None, Cookie(alias="session")] = None, ) -> uuid.UUID: - import logging - logger = logging.getLogger(__name__) - if not session_cookie: - logger.warning("No session cookie found in request") raise HTTPException(status_code=status.HTTP_401_UNAUTHORIZED, detail="missing session") - - logger.debug("Session cookie found: %s...", session_cookie[:20]) settings = Settings() try: @@ -36,7 +30,7 @@ async def get_current_user_id( async def get_current_user( - session_cookie: Annotated[str | None, Cookie()] = None, + session_cookie: Annotated[str | None, Cookie(alias="session")] = None, db_session: AsyncSession = Depends(get_db_session), ) -> User: if not session_cookie: