fix: logging-issues (#256)

Author: yxtay

Dockerfile

@@ -92,7 +92,7 @@ RUN apt-get update && \
     patchelf \
     && rm -rf /var/lib/apt/lists/*
 
-RUN uv pip install scons && \
+RUN uv pip install --no-cache-dir scons~=4.9 && \
     uv sync --group compile && \
     uv pip list
 

src/example_app/gunicorn.py

@@ -6,15 +6,14 @@
 import uvicorn.workers
 from gunicorn import config, glogging
 
-from .logger import InterceptHandler, get_logger
+from .logger import InterceptHandler
 from .settings import settings
 
 
 class GunicornLogger(glogging.Logger):
     def __init__(self: GunicornLogger, cfg: config.Config) -> None:
         super().__init__(cfg)
 
-        get_logger()
         for name in ["gunicorn.error", "gunicorn.access"]:
             logging.getLogger(name).handlers = [InterceptHandler(depth=1)]
 

src/example_app/logger.py

@@ -1,9 +1,7 @@
 from __future__ import annotations
 
-import atexit
 import inspect
 import logging
-import sys
 
 import loguru
 from loguru import logger
@@ -36,17 +34,6 @@ def emit(self: InterceptHandler, record: logging.LogRecord) -> None:
 
 
 def get_logger() -> loguru.Logger:
-    from .settings import settings
-
-    logger.remove()
-    logger.add(
-        sys.stderr, level=settings.loguru_level.upper(), enqueue=settings.loguru_enqueue
-    )
-    atexit.register(logger.complete)
-
-    logging.basicConfig(
-        level=settings.loguru_level.upper(), handlers=[InterceptHandler()]
-    )
     for name in ["uvicorn.access", "uvicorn.error"]:
         if name in logging.root.manager.loggerDict:
             _logger = logging.getLogger(name)

src/example_app/main.py

@@ -29,5 +29,4 @@ async def ok() -> Ok:
         host=settings.host,
         port=settings.port,
         reload=True,
-        log_config=None,
     )

src/example_app/settings.py

@@ -16,9 +16,8 @@ class Settings(BaseSettings):
     port: int = 8000
     web_concurrency: int = 2 * multiprocessing.cpu_count() + 1
 
-    # logging
-    loguru_level: str = "info"
-    loguru_enqueue: bool = True
+    # loguru
+    loguru_level: str = "INFO"
 
     model_config = SettingsConfigDict(env_file=".env")