Improvements from code review

Author: antonpirker

sentry_sdk/integrations/wsgi.py

@@ -356,13 +356,9 @@ def _finish_long_running_transaction(current_scope, isolation_scope):
     Triggered after MAX_TRANSACTION_DURATION_SECONDS have passed.
     """
     try:
-        transaction_duration = (
-            datetime.now(timezone.utc) - current_scope.transaction.start_timestamp
-        ).total_seconds()
-        if transaction_duration > MAX_TRANSACTION_DURATION_SECONDS:
-            with use_isolation_scope(isolation_scope):
-                with use_scope(current_scope):
-                    finish_running_transaction()
+        with use_isolation_scope(isolation_scope):
+            with use_scope(current_scope):
+                finish_running_transaction()
     except AttributeError:
         # transaction is not there anymore
         pass

sentry_sdk/tracing_utils.py

@@ -746,13 +746,13 @@ def get_current_span(scope=None):
 
 def finish_running_transaction(scope=None, exc_info=None, timer=None):
     # type: (Optional[sentry_sdk.Scope], Optional[ExcInfo], Optional[Timer]) -> None
+    if timer is not None:
+        timer.cancel()
+
     current_scope = scope or sentry_sdk.get_current_scope()
     if current_scope.transaction is not None and hasattr(
         current_scope.transaction, "_context_manager_state"
     ):
-        if timer is not None:
-            timer.cancel()
-
         if exc_info is not None:
             current_scope.transaction.__exit__(*exc_info)
         else: