fix: change SpanContext.traceFlags to mandatory (open-telemetry#818)

* fix: change SpanContext.traceFlags to mandatory According to spec SpanContext represents the W3C tracestate which includes traceId, spanId and traceFlags. As a side effect a new LinkContext types was added as links don't have traceFlags according to spec. * chore: review findings, rename TraceFlags.UNSAMPLED to NONE * fix: build * fix: tests * fix: correct merge Co-authored-by: Daniel Dyla <[email protected]>
dynatrace-oss-contrib · Mar 6, 2020 · 7bd62b7 · 7bd62b7
1 parent 09f881c
commit 7bd62b7
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/JaegerHttpTraceFormat.ts b/src/JaegerHttpTraceFormat.ts
@@ -58,9 +58,9 @@ export class JaegerHttpTraceFormat implements HttpTextFormat {
     const spanContext = getParentSpanContext(context);
     if (!spanContext) return;
 
-    const traceFlags = `0${(
-      spanContext.traceFlags || TraceFlags.UNSAMPLED
-    ).toString(16)}`;
+    const traceFlags = `0${(spanContext.traceFlags || TraceFlags.NONE).toString(
+      16
+    )}`;
 
     setter(
       carrier,