Skip to content

Commit 0783eb4

Browse files
fixed logging issue
1 parent 5f03bd6 commit 0783eb4

File tree

2 files changed

+6
-15
lines changed

2 files changed

+6
-15
lines changed

src/backend/settings.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,14 +41,14 @@ class _LoggingSettings(BaseSettings):
4141

4242
basic_logging_level: Literal["DEBUG", "INFO", "WARNING", "ERROR"] = "INFO"
4343
package_logging_level: Literal["DEBUG", "INFO", "WARNING", "ERROR"] = "WARNING"
44-
logging_packages: List[str] = []
44+
logging_packages: Optional[List[str]] = []
4545

4646
@field_validator("logging_packages", mode="before")
4747
@classmethod
48-
def split_logging_packages(cls, packages: str) -> List[str]:
48+
def split_logging_packages(cls, packages) -> Optional[List[str]]:
4949
if isinstance(packages, str) and len(packages.strip()) > 0:
5050
return [pkg.strip() for pkg in packages.split(",") if pkg.strip()]
51-
return []
51+
return None
5252

5353
def get_basic_log_level(self) -> int:
5454
"""Convert string log level to logging constant"""

src/backend/utils.py

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -165,17 +165,8 @@ def configure_logging(logging_settings):
165165
force=True # Override any existing configuration
166166
)
167167

168-
# Configure Azure package logging
169-
# Use custom packages if specified, otherwise use default Azure packages
170-
azure_loggers = logging_settings.logging_packages or [
171-
"azure.core.pipeline.policies.http_logging_policy",
172-
"azure.identity.aio._internal",
173-
"azure.monitor.opentelemetry.exporter.export._base"
174-
]
175-
176-
# Set Azure package loggers to the specified level
177168
azure_log_level = logging_settings.get_package_log_level()
178-
for logger_name in azure_loggers:
169+
for logger_name in logging_settings.logging_packages or []:
179170
logging.getLogger(logger_name).setLevel(azure_log_level)
180-
181-
logging.info(f"Logging configured - Basic: {logging_settings.basic_logging_level}, Azure packages: {logging_settings.package_logging_level}, Packages: {azure_loggers}")
171+
172+
logging.info(f"Logging configured - Basic: {logging_settings.basic_logging_level}, Azure packages: {logging_settings.package_logging_level}, Packages: {logging_settings.logging_packages}")

0 commit comments

Comments
 (0)