Fix DISABLE_AUTH check in get_current_user dependency

The auth dependency was requiring the Authorization header before
checking DISABLE_AUTH, causing API endpoints to fail in dev mode.
Now returns mock user immediately when DISABLE_AUTH=true.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
michael 2025-12-18 10:15:37 -06:00
parent 99af0164e6
commit 0742cc0aa4

View file

@ -6,6 +6,7 @@ Provides dependency functions for securing REST endpoints with Azure AD token ve
from typing import Optional
from fastapi import Header, HTTPException, status
from app.config import settings
from app.services.auth_service import verify_access_token
@ -27,6 +28,10 @@ async def get_current_user(authorization: Optional[str] = Header(None)) -> dict:
Raises:
HTTPException: 401 if token is missing or invalid
"""
# If auth is disabled, return mock user immediately
if settings.DISABLE_AUTH:
return {"sub": "dev-user", "name": "Development User", "preferred_username": "dev@localhost"}
if not authorization:
raise HTTPException(
status_code=status.HTTP_401_UNAUTHORIZED,