Remove setting HTTP_ROUTE attribute in frontend

As per the OpenTelemetry HTTP Server semantic conventions, `HTTP.ROUTE` is meant to be a low cardinality route template. The URI cannot substitute it. This patch remove the usage of the HTTP_ROUTE semantic attribute.
open-telemetry · Nov 8, 2022 · 4b9048b · 4b9048b
1 parent 603d62a
commit 4b9048b
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 2 deletions.
diff --git a/docs/services/frontend.md b/docs/services/frontend.md
@@ -75,7 +75,6 @@ application context.
             "app.synthetic_request": true,
             [SemanticAttributes.HTTP_TARGET]: target,
             [SemanticAttributes.HTTP_STATUS_CODE]: response.statusCode,
-            [SemanticAttributes.HTTP_ROUTE]: url,
             [SemanticAttributes.HTTP_METHOD]: method,
             [SemanticAttributes.HTTP_USER_AGENT]: headers['user-agent'] || '',
             [SemanticAttributes.HTTP_URL]: `${headers.host}${url}`,

diff --git a/src/frontend/utils/telemetry/InstrumentationMiddleware.ts b/src/frontend/utils/telemetry/InstrumentationMiddleware.ts
@@ -22,7 +22,6 @@ const InstrumentationMiddleware = (handler: NextApiHandler): NextApiHandler => {
                     "app.synthetic_request": true,
                     [SemanticAttributes.HTTP_TARGET]: target,
                     [SemanticAttributes.HTTP_STATUS_CODE]: response.statusCode,
-                    [SemanticAttributes.HTTP_ROUTE]: url,
                     [SemanticAttributes.HTTP_METHOD]: method,
                     [SemanticAttributes.HTTP_USER_AGENT]: headers['user-agent'] || '',
                     [SemanticAttributes.HTTP_URL]: `${headers.host}${url}`,