Move ACCESS_LOG setting to config and use it

Add ACCESS_LOG_ENABLED to src/config/settings.py (defaults to true; accepts true/1/yes) with a comment explaining use cases (disable when fronted by a load balancer or to reduce CI noise). Update src/main.py to import and use ACCESS_LOG_ENABLED for uvicorn's access_log, and remove the now-unused local getenv/os usage. Centralizes the access-log configuration.

Author: edwinjosechittilappilly

src/config/settings.py

@@ -90,6 +90,11 @@
     "yes",
 )
 
+# Whether uvicorn emits an access log line per HTTP request. On by
+# default; flip via ACCESS_LOG=false (e.g. when fronted by a load balancer
+# that already logs requests, or to reduce log noise in CI).
+ACCESS_LOG_ENABLED = os.getenv("ACCESS_LOG", "true").lower() in ("true", "1", "yes")
+
 # Number of uvicorn worker processes to allow. Multi-worker is currently
 # unsupported because the RBAC permission cache and the OAuth-subject→DB-id
 # cache are per-process; the lifespan startup hook hard-fails if this is >1

src/main.py

@@ -22,14 +22,14 @@
 
 import asyncio
 import atexit
-import os
 
 # Re-exported so tests that monkeypatch "main.httpx.AsyncClient" keep working.
 # Module attributes in Python are shared singletons, so the patch propagates
 # to services/default_docs_service.py which calls httpx.AsyncClient directly.
 import httpx  # noqa: F401
 
 from app.factory import create_app
+from config.settings import ACCESS_LOG_ENABLED
 from services.default_docs_service import (
     _get_remote_docs_signature,
     _should_use_url_default_docs_ingest,
@@ -87,14 +87,12 @@ def cleanup():
 
     app = asyncio.run(create_app())
 
-    access_log = os.getenv("ACCESS_LOG", "true").lower() == "true"
-
     uvicorn.run(
         app,
         workers=1,
         host="0.0.0.0",
         port=8000,
         reload=False,
-        access_log=access_log,
+        access_log=ACCESS_LOG_ENABLED,
         log_config=None,
     )