Skip to content

Commit 5df3514

Browse files
authored
Reactor: avoid attaching context if it's already attached (#11393)
Reactor: avoid attaching context if it's already attached Co-authored-by: andrea.marziali <andrea.marziali@datadoghq.com>
1 parent ef9d47c commit 5df3514

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

dd-java-agent/instrumentation/reactive-streams-1.0/src/main/java/datadog/trace/instrumentation/reactivestreams/PublisherInstrumentation.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ public static ContextScope onSubscribe(
6363
final Context current =
6464
InstrumentationContext.get(Subscriber.class, Context.class)
6565
.putIfAbsent(s, context != null ? context : activeContext);
66-
if (current != null) {
66+
if (current != null && activeContext != current) {
6767
return current.attach();
6868
}
6969

dd-java-agent/instrumentation/reactive-streams-1.0/src/main/java/datadog/trace/instrumentation/reactivestreams/SubscriberInstrumentation.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ public static ContextScope before(@Advice.This final Subscriber self) {
5656
return null;
5757
}
5858
final Context context = InstrumentationContext.get(Subscriber.class, Context.class).get(self);
59-
return context == null ? null : context.attach();
59+
return context == null || context == currentContext ? null : context.attach();
6060
}
6161

6262
@Advice.OnMethodExit(onThrowable = Throwable.class, suppress = Throwable.class)

0 commit comments

Comments
 (0)