consider regex route paths

Author: s1gr1d

plugins/node/opentelemetry-instrumentation-express/src/utils.ts

@@ -266,6 +266,16 @@ export function getActualMatchedRoute(req: {
     return constructedRoute;
   }
 
+  // For RegExp routes or route arrays, return the constructed route
+  // This handles the case where the route is defined using RegExp or an array
+  if (constructedRoute.includes('/') &&
+      (constructedRoute.includes(',') ||
+       constructedRoute.includes('\\') ||
+       constructedRoute.includes('*') ||
+       constructedRoute.includes('['))) {
+    return constructedRoute;
+  }
+
   // Ensure route starts with '/' if it doesn't already
   const normalizedRoute = constructedRoute.startsWith('/')
     ? constructedRoute

plugins/node/opentelemetry-instrumentation-express/test/express.test.ts

@@ -103,7 +103,7 @@ describe('ExpressInstrumentation', () => {
 
           for (const span of spans) {
             assert.strictEqual(
-              span.attributes[SEMATTRS_HTTP_ROUTE],
+              span.attributes[ATTR_HTTP_ROUTE],
               undefined, // none of the spans have the HTTP route attribute
               `Span "${span.name}" should not have HTTP route attribute for non-existing route`
             );