updated config code

Author: AjitPadhi-Microsoft

infra/main.bicep

@@ -1503,7 +1503,6 @@ module webSite 'modules/web-sites.bicep' = {
           AZURE_BASIC_LOGGING_LEVEL: 'INFO'
           AZURE_PACKAGE_LOGGING_LEVEL: 'WARNING'
           AZURE_LOGGING_PACKAGES: ''
-
         }
         // WAF aligned configuration for Monitoring
         applicationInsightResourceId: enableMonitoring ? applicationInsights!.outputs.resourceId : null

src/backend/app_kernel.py

@@ -65,8 +65,11 @@ async def lifespan(app: FastAPI):
 
 # Configure Azure package logging levels
 azure_level = getattr(logging, config.AZURE_PACKAGE_LOGGING_LEVEL.upper(), logging.WARNING)
-for logger_name in config.AZURE_LOGGING_PACKAGES:
-    logging.getLogger(logger_name).setLevel(azure_level)
+# Parse comma-separated logging packages
+if config.AZURE_LOGGING_PACKAGES:
+    packages = [pkg.strip() for pkg in config.AZURE_LOGGING_PACKAGES.split(",") if pkg.strip()]
+    for logger_name in packages:
+        logging.getLogger(logger_name).setLevel(azure_level)
 
 logging.getLogger("opentelemetry.sdk").setLevel(logging.ERROR)
 

src/backend/common/config/app_config.py

@@ -76,7 +76,7 @@ def __init__(self):
         # Logging settings
         self.AZURE_BASIC_LOGGING_LEVEL = self._get_optional("AZURE_BASIC_LOGGING_LEVEL", "INFO")
         self.AZURE_PACKAGE_LOGGING_LEVEL = self._get_optional("AZURE_PACKAGE_LOGGING_LEVEL", "WARNING")
-        self.AZURE_LOGGING_PACKAGES = self._get_logging_packages("AZURE_LOGGING_PACKAGES")
+        self.AZURE_LOGGING_PACKAGES = self._get_optional("AZURE_LOGGING_PACKAGES")
 
         # Optional MCP server endpoint (for local MCP server or remote)
         # Example: http://127.0.0.1:8000/mcp
@@ -196,25 +196,6 @@ def _get_bool(self, name: str) -> bool:
         """
         return name in os.environ and os.environ[name].lower() in ["true", "1"]
 
-    def _get_logging_packages(self, name: str) -> list[str]:
-        """Get a list of logging package names from environment variables.
-
-        Args:
-            name: The name of the environment variable containing comma-separated package names
-
-        Returns:
-            A list of package names, or empty list if not specified
-        """
-        try:
-            env_value = os.environ.get(name, "").strip()
-            if env_value:
-                packages = [pkg.strip() for pkg in env_value.split(",") if pkg.strip()]
-                return packages
-            return []
-        except (AttributeError, ValueError) as e:
-            logging.warning(f"Error parsing {name}: {e}. Using empty list.")
-            return []
-
     def get_cosmos_database_client(self):
         """Get a Cosmos DB client for the configured database.