Fix shouldTrace decision based on sampling decision from edge

vtex · Sep 1, 2023 · 2783499 · 2783499
1 parent 750d149
commit 2783499
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/service/tracing/tracingMiddlewares.ts b/src/service/tracing/tracingMiddlewares.ts
@@ -152,6 +152,6 @@ export const traceUserLandRemainingPipelineMiddleware = () => {
 }
 function shouldTrace(ctx: ServiceContext, rootSpan: SpanContext | undefined) {
   // Should trace if path isnt blacklisted and tracing decision came from the edge
-  return !PATHS_BLACKLISTED_FOR_TRACING.includes(ctx.request.path) && rootSpan != null
+  return !PATHS_BLACKLISTED_FOR_TRACING.includes(ctx.request.path) && ((rootSpan as any).isSampled?.() ?? false)
 }