Issue 6037 - Wrapping "invalid" SpanContexts in Span does not preserv…

…e SpanContext (#6044)
open-telemetry · Dec 8, 2023 · 9f3456f · 9f3456f
1 parent 3bb221f
commit 9f3456f
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/api/all/src/main/java/io/opentelemetry/api/trace/Span.java b/api/all/src/main/java/io/opentelemetry/api/trace/Span.java
@@ -74,16 +74,13 @@ static Span getInvalid() {
   /**
    * Returns a non-recording {@link Span} that holds the provided {@link SpanContext} but has no
    * functionality. It will not be exported and all tracing operations are no-op, but it can be used
-   * to propagate a valid {@link SpanContext} downstream.
+   * to propagate a {@link SpanContext} downstream.
    */
   static Span wrap(SpanContext spanContext) {
     if (spanContext == null) {
       ApiUsageLogger.log("context is null");
       return getInvalid();
     }
-    if (!spanContext.isValid()) {
-      return getInvalid();
-    }
     return PropagatedSpan.create(spanContext);
   }