Fix review issues for NAVI_AUTH

Fork: 0

root / navi-1

Browse code Fix review issues for NAVI_AUTH_ENABLED feature - Return 503 from /auth/login and /auth/callback when NAVI_AUTH_ENABLED=false - Return model_copy() of _ANONYMOUS_USER to prevent accidental mutation - Clean up __import__(datetime) in auth DDL - Reorder FakeChatSession definition before use in tests - Remove dead monkeypatch code in no-auth integration fixture - Add unit test for get_current_user_ws in no-auth mode - Add integration tests rejecting OAuth endpoints in no-auth mode Co-Authored-By: Claude <noreply@anthropic.com> feature/navi-code master
1 parent 4923a2b commit de093d8e52926d69b708725a49cc510f9cfa3cca Eugene Sukhodolskiy authored 2 days ago

Browse code

Fix review issues for NAVI_AUTH_ENABLED feature

- Return 503 from /auth/login and /auth/callback when NAVI_AUTH_ENABLED=false
- Return model_copy() of _ANONYMOUS_USER to prevent accidental mutation
- Clean up __import__(datetime) in auth DDL
- Reorder FakeChatSession definition before use in tests
- Remove dead monkeypatch code in no-auth integration fixture
- Add unit test for get_current_user_ws in no-auth mode
- Add integration tests rejecting OAuth endpoints in no-auth mode

Co-Authored-By: Claude <noreply@anthropic.com>

feature/navi-code master

1 parent 4923a2b commit de093d8e52926d69b708725a49cc510f9cfa3cca

Eugene Sukhodolskiy authored 2 days ago

Patch

Unified Split

Showing 5 changed files

Ignore Space Show notes View navi/api/routes/auth.py

Ignore Space Show notes View navi/auth/_ddl.py

Ignore Space Show notes View navi/auth/deps.py

Ignore Space Show notes View tests/integration/test_auth_disabled.py

Ignore Space Show notes View tests/unit/auth/test_deps.py

Show line notes below