Revert "try wsgi fix"

This reverts commit dba7521.

Author: felix-hilden

instrumentation/opentelemetry-instrumentation-wsgi/src/opentelemetry/instrumentation/wsgi/__init__.py

@@ -706,13 +706,16 @@ def __call__(
                     self._sem_conv_opt_in_mode,
                 )
                 iterable = self.wsgi(environ, start_response)
-                return _iterate_and_close_with_span(iterable, span, token)
+                return _end_span_after_iterating(iterable, span, token)
         except Exception as ex:
             if _report_new(self._sem_conv_opt_in_mode):
                 req_attrs[ERROR_TYPE] = type(ex).__qualname__
                 if span.is_recording():
                     span.set_attribute(ERROR_TYPE, type(ex).__qualname__)
                 span.set_status(Status(StatusCode.ERROR, str(ex)))
+            span.end()
+            if token is not None:
+                context.detach(token)
             raise
         finally:
             duration_s = default_timer() - start
@@ -736,16 +739,13 @@ def __call__(
                     context=active_metric_ctx,
                 )
             self.active_requests_counter.add(-1, active_requests_count_attrs)
-            span.end()
-            if token is not None:
-                context.detach(token)
 
 
 # Put this in a subfunction to not delay the call to the wrapped
 # WSGI application (instrumentation should change the application
 # behavior as little as possible).
-def _iterate_and_close_with_span(
-    iterable: Iterable[T], span: trace.Span
+def _end_span_after_iterating(
+    iterable: Iterable[T], span: trace.Span, token: object
 ) -> Iterable[T]:
     try:
         with trace.use_span(span):
@@ -754,6 +754,9 @@ def _iterate_and_close_with_span(
         close = getattr(iterable, "close", None)
         if close:
             close()
+        span.end()
+        if token is not None:
+            context.detach(token)
 
 
 # TODO: inherit from opentelemetry.instrumentation.propagators.Setter