fix: capture exceptions at the span level

Author: logaretm

packages/nuxt/src/runtime/hooks/instrumentMiddlewareHandler.ts

@@ -1,8 +1,11 @@
 import {
+  captureException,
   debug,
   flushIfServerless,
   getDefaultIsolationScope,
   getIsolationScope,
+  SEMANTIC_ATTRIBUTE_SENTRY_OP,
+  SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN,
   SEMANTIC_ATTRIBUTE_SENTRY_SOURCE,
   SPAN_STATUS_ERROR,
   SPAN_STATUS_OK,
@@ -23,18 +26,18 @@ export function instrumentMiddlewareHandler(handler: EventHandler, fileName: str
 
     debug.log(`Sentry middleware: ${fileName} handling ${middlewarePath}`);
 
+    const origin = 'auto.http.nuxt';
     const isolationScope = getIsolationScope();
     const newIsolationScope = isolationScope === getDefaultIsolationScope() ? isolationScope.clone() : isolationScope;
 
     return withIsolationScope(newIsolationScope, async () => {
       return startSpan(
         {
-          name: `middleware.${fileName}`,
-          op: 'middleware.nitro',
+          name: `${fileName}`,
           attributes: {
-            [SEMANTIC_ATTRIBUTE_SENTRY_SOURCE]: 'custom',
-            'middleware.name': fileName,
-            'middleware.path': middlewarePath,
+            [SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'http.server.middleware',
+            [SEMANTIC_ATTRIBUTE_SENTRY_SOURCE]: 'route',
+            [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: origin,
           },
         },
         async span => {
@@ -45,6 +48,15 @@ export function instrumentMiddlewareHandler(handler: EventHandler, fileName: str
           } catch (error) {
             span.setStatus({ code: SPAN_STATUS_ERROR, message: 'internal_error' });
             span.recordException(error);
+            captureException(error, {
+              mechanism: {
+                handled: false,
+                type: origin,
+              },
+            });
+
+            span.end();
+            // Re-throw the error to be handled by the caller
             throw error;
           } finally {
             await flushIfServerless();