simplify try logic

Author: quinnmil

scout_apm_logging/handler.py

@@ -60,57 +60,52 @@ def setup_logging(self):
         )
 
     def emit(self, record):
-        try:
-            # Initialize here to ensure that required configuration variables are loaded
-            if not ScoutOtelHandler._class_initialized:
-                try:
-                    self._initialize()
-                except Exception as e:
-                    print(f"Failed to initialize ScoutOtelHandler: {e}")
-                    return
-
-            if getattr(self._handling_log, "value", False):
-                # We're already handling a log message, don't get the TrackedRequest
-                return self.otel_handler.emit(record)
-
-            self._handling_log.value = True
-            scout_request = TrackedRequest.instance()
-
-            if scout_request:
-                operation_detail = get_operation_detail(scout_request)
-                if operation_detail:
-                    setattr(
-                        record,
-                        operation_detail.entrypoint_attribute,
-                        operation_detail.name,
-                    )
-
-                # Add Scout-specific attributes to the log record
-                record.scout_request_id = scout_request.request_id
-                record.scout_start_time = scout_request.start_time.isoformat()
-                # Add duration if the request is completed
-                if scout_request.end_time:
-                    record.scout_end_time = scout_request.end_time.isoformat()
-                    record.scout_duration = (
-                        scout_request.end_time - scout_request.start_time
-                    ).total_seconds()
-
-                setattr(record, "service.name", self.service_name)
-
-                # Add tags
-                for key, value in scout_request.tags.items():
-                    setattr(record, f"scout_tag_{key}", value)
-
-                # Add the current span's operation if available
-                current_span = scout_request.current_span()
-                if current_span:
-                    record.scout_current_operation = current_span.operation
-
-            self.otel_handler.emit(record)
-        except Exception as e:
-            print(f"Error in ScoutOtelHandler.emit: {e}")
-        finally:
-            self._handling_log.value = False
+        if not ScoutOtelHandler._class_initialized:
+            try:
+                self._initialize()
+            except Exception as e:
+                print(f"Failed to initialize ScoutOtelHandler: {e}")
+                return
+
+        if getattr(self._handling_log, "value", False):
+            # We're already handling a log message, don't get the TrackedRequest
+            return self.otel_handler.emit(record)
+
+        self._handling_log.value = True
+        scout_request = TrackedRequest.instance()
+
+        if scout_request:
+            operation_detail = get_operation_detail(scout_request)
+            if operation_detail:
+                setattr(
+                    record,
+                    operation_detail.entrypoint_attribute,
+                    operation_detail.name,
+                )
+
+            # Add Scout-specific attributes to the log record
+            record.scout_request_id = scout_request.request_id
+            record.scout_start_time = scout_request.start_time.isoformat()
+            # Add duration if the request is completed
+            if scout_request.end_time:
+                record.scout_end_time = scout_request.end_time.isoformat()
+                record.scout_duration = (
+                    scout_request.end_time - scout_request.start_time
+                ).total_seconds()
+
+            setattr(record, "service.name", self.service_name)
+
+            # Add tags
+            for key, value in scout_request.tags.items():
+                setattr(record, f"scout_tag_{key}", value)
+
+            # Add the current span's operation if available
+            current_span = scout_request.current_span()
+            if current_span:
+                record.scout_current_operation = current_span.operation
+
+        self.otel_handler.emit(record)
+        self._handling_log.value = False
 
     def close(self):
         if self.logger_provider: