From 512031bf888d9f1ce7365b3dd5d4b4b221eb3bdc Mon Sep 17 00:00:00 2001 From: MrWaradana Date: Thu, 12 Feb 2026 14:48:03 +0700 Subject: [PATCH] fix loggin level no error --- src/logging.py | 7 ++++++- src/main.py | 5 ++++- 2 files changed, 10 insertions(+), 2 deletions(-) diff --git a/src/logging.py b/src/logging.py index a7f31af..42f869c 100644 --- a/src/logging.py +++ b/src/logging.py @@ -119,11 +119,16 @@ def configure_logging(): root_logger.addHandler(handler) # Reconfigure uvicorn loggers to use our JSON formatter - for logger_name in ["uvicorn", "uvicorn.access", "uvicorn.error", "fastapi"]: + for logger_name in ["uvicorn", "uvicorn.error", "fastapi"]: logger = logging.getLogger(logger_name) logger.handlers = [] logger.propagate = True + # Disable uvicorn access logs as we handle request logging in our middleware + access_logger = logging.getLogger("uvicorn.access") + access_logger.handlers = [] + access_logger.propagate = False + # sometimes the slack client can be too verbose logging.getLogger("slack_sdk.web.base_client").setLevel(logging.CRITICAL) diff --git a/src/main.py b/src/main.py index b5f803b..8f8f8c1 100644 --- a/src/main.py +++ b/src/main.py @@ -71,7 +71,10 @@ async def db_session_middleware(request: Request, call_next): collector_session = async_scoped_session(collector_async_session, scopefunc=get_request_id) request.state.collector_db = collector_session() response = await call_next(request) - log.info(f"Request completed: {response.status_code}") + if response.status_code >= 400: + log.error(f"Request completed: {response.status_code}") + else: + log.info(f"Request completed: {response.status_code}") except Exception as e: log.error(f"Request failed: {type(e).__name__} - {str(e)}") raise e from None