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.
This commit is contained in:
@@ -151,9 +151,7 @@ async def me(
|
|||||||
session_cookie: str | None = Cookie(default=None, alias="session"),
|
session_cookie: str | None = Cookie(default=None, alias="session"),
|
||||||
session: AsyncSession = Depends(get_db_session),
|
session: AsyncSession = Depends(get_db_session),
|
||||||
) -> dict[str, Any]:
|
) -> dict[str, Any]:
|
||||||
logger.info("/auth/me called, session_cookie present: %s", bool(session_cookie))
|
|
||||||
if not session_cookie:
|
if not session_cookie:
|
||||||
logger.warning("/auth/me: no session cookie")
|
|
||||||
raise HTTPException(status_code=status.HTTP_401_UNAUTHORIZED, detail="missing session")
|
raise HTTPException(status_code=status.HTTP_401_UNAUTHORIZED, detail="missing session")
|
||||||
|
|
||||||
settings = Settings()
|
settings = Settings()
|
||||||
|
|||||||
@@ -16,17 +16,11 @@ async def get_db_session():
|
|||||||
|
|
||||||
|
|
||||||
async def get_current_user_id(
|
async def get_current_user_id(
|
||||||
session_cookie: Annotated[str | None, Cookie()] = None,
|
session_cookie: Annotated[str | None, Cookie(alias="session")] = None,
|
||||||
) -> uuid.UUID:
|
) -> uuid.UUID:
|
||||||
import logging
|
|
||||||
logger = logging.getLogger(__name__)
|
|
||||||
|
|
||||||
if not session_cookie:
|
if not session_cookie:
|
||||||
logger.warning("No session cookie found in request")
|
|
||||||
raise HTTPException(status_code=status.HTTP_401_UNAUTHORIZED, detail="missing session")
|
raise HTTPException(status_code=status.HTTP_401_UNAUTHORIZED, detail="missing session")
|
||||||
|
|
||||||
logger.debug("Session cookie found: %s...", session_cookie[:20])
|
|
||||||
|
|
||||||
settings = Settings()
|
settings = Settings()
|
||||||
try:
|
try:
|
||||||
payload = decode_session_cookie(settings=settings, cookie_value=session_cookie)
|
payload = decode_session_cookie(settings=settings, cookie_value=session_cookie)
|
||||||
@@ -36,7 +30,7 @@ async def get_current_user_id(
|
|||||||
|
|
||||||
|
|
||||||
async def get_current_user(
|
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),
|
db_session: AsyncSession = Depends(get_db_session),
|
||||||
) -> User:
|
) -> User:
|
||||||
if not session_cookie:
|
if not session_cookie:
|
||||||
|
|||||||
Reference in New Issue
Block a user