fix: remove setting http.route in http span attributes (#2494)

Author: mustafain117

experimental/packages/opentelemetry-instrumentation-http/src/utils.ts

@@ -453,7 +453,6 @@ export const getIncomingRequestAttributes = (
   }
 
   if (requestUrl) {
-    attributes[SemanticAttributes.HTTP_ROUTE] = requestUrl.pathname || '/';
     attributes[SemanticAttributes.HTTP_TARGET] = requestUrl.pathname || '/';
   }
 

experimental/packages/opentelemetry-instrumentation-http/test/functionals/utils.test.ts

@@ -466,6 +466,21 @@ describe('Utility', () => {
     });
   });
 
+  describe('getIncomingRequestAttributes()', () => {
+    it('should not set http.route in http span attributes', () => {
+      const request = {
+        url: 'http://hostname/user/:id',
+        method: 'GET'
+      } as IncomingMessage;
+      request.headers = {
+        'user-agent': 'chrome',
+        'x-forwarded-for': '<client>, <proxy1>, <proxy2>'
+      }
+      const attributes = utils.getIncomingRequestAttributes(request, { component: 'http'})
+      assert.strictEqual(attributes[SemanticAttributes.HTTP_ROUTE], undefined)
+    });
+  });
+  
   describe('headers to span attributes capture', () => {
     let span: Span;