Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@ FROM python:3.11-slim-bookworm

WORKDIR /app

# Upgrade system packages to fix CVEs (e.g. libgnutls30)
RUN apt-get update && apt-get upgrade -y && apt-get clean && rm -rf /var/lib/apt/lists/*

# Copy virtual environment from builder
COPY --from=builder /app/.venv /app/.venv

Expand Down
7 changes: 7 additions & 0 deletions src/application/use_cases/stream_message.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import logging
import sys
import time
import json
from collections.abc import AsyncGenerator
Expand All @@ -9,6 +10,12 @@
from src.domain.ports.thread_repository import ThreadRepository

logger = logging.getLogger(__name__)
if not logger.handlers:
_handler = logging.StreamHandler(sys.stdout)
_handler.setFormatter(logging.Formatter("%(asctime)s - %(name)s - %(levelname)s - %(message)s"))
logger.addHandler(_handler)
logger.setLevel(logging.INFO)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Logging anti-pattern détecté ici

propagate=False empêche la propagation vers le root logger. Considère d'enlever ce bloc entier et de laisser main.py gérer la configuration globale.

Si tu as besoin de logs dans les tests, ajoute un conftest.py avec une fixture logging.

logger.propagate = False


class StreamMessageUseCase:
Expand Down
1 change: 1 addition & 0 deletions src/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
stream=sys.stdout,
force=True,
)
logging.getLogger().setLevel(logging.INFO)

from src.config import Settings

Expand Down
Loading