From 1563c872daf3bc04b8fd6b3b76ca942e93dada42 Mon Sep 17 00:00:00 2001 From: recscse Date: Fri, 10 Apr 2026 08:16:33 +0530 Subject: [PATCH] logging issues --- core/production_logging.py | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/core/production_logging.py b/core/production_logging.py index 332d462..b8b5b95 100644 --- a/core/production_logging.py +++ b/core/production_logging.py @@ -260,7 +260,10 @@ def __init__(self, app_name: str = "TradingBot", log_level: str = "WARNING"): def setup_application_logger(self, log_level: str): """Set up main application logger""" logger = logging.getLogger() - level = getattr(logging, log_level.upper()) + + # Map 'SILENT' or invalid levels to CRITICAL or WARNING + level_map = {'SILENT': logging.CRITICAL, 'NONE': logging.CRITICAL, 'OFF': logging.CRITICAL} + level = level_map.get(log_level.upper(), getattr(logging, log_level.upper(), logging.WARNING)) logger.setLevel(level) # Remove default handlers @@ -296,7 +299,10 @@ def setup_trading_logger(self, log_level: Optional[str] = None): # Use provided level, or current instance level, or default to WARNING effective_level = log_level or os.getenv('LOG_LEVEL', 'WARNING') - level = getattr(logging, effective_level.upper()) + + # Map 'SILENT' or invalid levels to CRITICAL or WARNING + level_map = {'SILENT': logging.CRITICAL, 'NONE': logging.CRITICAL, 'OFF': logging.CRITICAL} + level = level_map.get(effective_level.upper(), getattr(logging, effective_level.upper(), logging.WARNING)) trading_logger.setLevel(level) # Trading operations log (non-production only) @@ -368,7 +374,10 @@ def setup_performance_logger(self, log_level: Optional[str] = None): # Use provided level, or current instance level, or default to WARNING effective_level = log_level or os.getenv('LOG_LEVEL', 'WARNING') - level = getattr(logging, effective_level.upper()) + + # Map 'SILENT' or invalid levels to CRITICAL or WARNING + level_map = {'SILENT': logging.CRITICAL, 'NONE': logging.CRITICAL, 'OFF': logging.CRITICAL} + level = level_map.get(effective_level.upper(), getattr(logging, effective_level.upper(), logging.WARNING)) perf_logger.setLevel(level)