Remove setting HTTP_ROUTE attribute in frontend (open-telemetry#566)

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.
jmichalak9 · Nov 11, 2022 · 7b16bbe · 7b16bbe
1 parent 48bd85d
commit 7b16bbe
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}`,