From 62a04c7a7d640393c31ae357131bfca766c1f3fb Mon Sep 17 00:00:00 2001 From: kyusyukeigo <694454+anchan828@users.noreply.github.com> Date: Sat, 6 Nov 2021 00:22:38 +0900 Subject: [PATCH] feat: report `path` and `route` with fastify backend in nestjs-core instrumentation (#714) Co-authored-by: Rauno Viskus --- .../src/instrumentation.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plugins/node/opentelemetry-instrumentation-nestjs-core/src/instrumentation.ts b/plugins/node/opentelemetry-instrumentation-nestjs-core/src/instrumentation.ts index c94c2ec0a3f..9aa1df479ab 100644 --- a/plugins/node/opentelemetry-instrumentation-nestjs-core/src/instrumentation.ts +++ b/plugins/node/opentelemetry-instrumentation-nestjs-core/src/instrumentation.ts @@ -187,8 +187,8 @@ function createWrapCreateHandler(tracer: api.Tracer, moduleVersion?: string) { [AttributeNames.VERSION]: moduleVersion, [AttributeNames.TYPE]: NestType.REQUEST_CONTEXT, [SemanticAttributes.HTTP_METHOD]: req.method, - [SemanticAttributes.HTTP_URL]: req.originalUrl, - [SemanticAttributes.HTTP_ROUTE]: req.route.path, + [SemanticAttributes.HTTP_URL]: req.originalUrl || req.url, + [SemanticAttributes.HTTP_ROUTE]: req.route?.path || req.routerPath, [AttributeNames.CONTROLLER]: instanceName, [AttributeNames.CALLBACK]: callbackName, },