Skip to content

Commit 6f4ab09

Browse files
authored
Merge pull request #4002 from froque/4001_fix_opentelemetry_parent_context
Replace Opentelemetry root with current to reuse existing trace.
2 parents 911eb1d + aa78f93 commit 6f4ab09

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

modules/jooby-opentelemetry/src/main/java/io/jooby/internal/opentelemetry/DefaultOtelContextExtractor.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
*/
66
package io.jooby.internal.opentelemetry;
77

8-
import static io.opentelemetry.context.Context.root;
8+
import static io.opentelemetry.context.Context.current;
99

1010
import org.jspecify.annotations.NonNull;
1111

@@ -30,7 +30,7 @@ public DefaultOtelContextExtractor(OpenTelemetry otel) {
3030
// 2. Secondary: If middleware is missing, manually parse the W3C headers
3131
var propagator = otel.getPropagators().getTextMapPropagator();
3232
// Extracts W3C headers (if present) or returns Context.current() as a safe fallback
33-
result = propagator.extract(root(), ctx, Headers.INSTANCE);
33+
result = propagator.extract(current(), ctx, Headers.INSTANCE);
3434
// Cache it to avoid re-parsing headers on subsequent calls in the same request
3535
ctx.setAttribute(Context.class.getName(), result);
3636
}

0 commit comments

Comments
 (0)