Logging configuration for Application Insights

Author: swebberuk

manage_breast_screening/config/settings.py

@@ -305,29 +305,12 @@ def list_env(key):
     "loggers": {
         "django": {
             "level": "INFO",
-            "handlers": ["console"],
-            "propagate": False,
         },
         "django.request": {
-            "handlers": ["console"],
-            "level": "INFO",
-            "propagate": False,
             "filters": ["suppress_duplicate_exceptions"],
         },
         "django.db.backends": {
             "level": "DEBUG" if LOG_QUERIES else "INFO",
-            "handlers": ["console"],
-            "propagate": False,
-        },
-        "django.server": {
-            "level": "INFO",
-            "handlers": ["console"],
-            "propagate": False,
-        },
-        "django.utils.autoreload": {
-            "level": "INFO",
-            "handlers": ["console"],
-            "propagate": False,
         },
         "faker": {"level": "INFO"},
         "msal": {"level": "INFO"},

manage_breast_screening/core/services/application_insights_logging.py

@@ -11,7 +11,7 @@ def __init__(self) -> None:
         self.logger_name = os.getenv(
             "APPLICATIONINSIGHTS_LOGGER_NAME", "insights-logger"
         )
-        os.environ.setdefault("OTEL_SERVICE_NAME", self.logger_name)
+        os.environ.setdefault("OTEL_SERVICE_NAME", "manage-breast-screening")
         self.logger = self.getLogger()
 
     def configure_azure_monitor(self):
@@ -20,10 +20,7 @@ def configure_azure_monitor(self):
         ):
             # Configure OpenTelemetry to use Azure Monitor with the
             # APPLICATIONINSIGHTS_CONNECTION_STRING environment variable.
-            configure_azure_monitor(
-                # Set the namespace for the logger in which you would like to collect telemetry for if you are collecting logging telemetry. This is imperative so you do not collect logging telemetry from the SDK itself.
-                logger_name=self.logger_name,
-            )
+            configure_azure_monitor()
         else:
             default_logger = logging.getLogger(__name__)
             default_logger.info("Application Insights logging not enabled")

manage_breast_screening/core/tests/services/test_application_insights_logging.py

@@ -24,7 +24,7 @@ def test_configures_azure_monitor_when_env_correct(
         monkeypatch.setenv("APPLICATIONINSIGHTS_IS_ENABLED", "True")
         monkeypatch.setenv("APPLICATIONINSIGHTS_CONNECTION_STRING", "something")
         ApplicationInsightsLogging().configure_azure_monitor()
-        mock_configure_azure.assert_called_with(logger_name="insights-logger")
+        mock_configure_azure.assert_called_with()
 
     def test_does_not_configure_if_flag_not_enabled(
         self, mock_logging, mock_configure_azure, monkeypatch