fix: remove setting http.route in http span attributes

open-telemetry · Sep 23, 2021 · 5e7cd47 · 5e7cd47
1 parent 6a78db1
commit 5e7cd47
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/experimental/packages/opentelemetry-instrumentation-http/src/utils.ts b/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 || '/';
   }
 

diff --git a/experimental/packages/opentelemetry-instrumentation-http/test/functionals/utils.test.ts b/experimental/packages/opentelemetry-instrumentation-http/test/functionals/utils.test.ts
@@ -465,4 +465,19 @@ describe('Utility', () => {
       verifyValueInAttributes(attributes, undefined, 1200);
     });
   });
+
+  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)
+    });
+  })
 });