Fix test_traceparent_ows_handling

psx95 · psx95 · commit 241485f4e92f · 2024-10-30T21:54:04.000Z
diff --git a/api/include/opentelemetry/trace/propagation/http_trace_context.h b/api/include/opentelemetry/trace/propagation/http_trace_context.h
@@ -182,7 +182,8 @@ class HttpTraceContext : public context::propagation::TextMapPropagator
 
   static SpanContext ExtractImpl(const context::propagation::TextMapCarrier &carrier)
   {
-    nostd::string_view trace_parent = carrier.Get(kTraceParent);
+    // Get trace_parent after trimming the leading and trailing whitespaces
+    nostd::string_view trace_parent = detail::TrimString(carrier.Get(kTraceParent));
     nostd::string_view trace_state  = carrier.Get(kTraceState);
     if (trace_parent == "")
     {

Original file line number	Diff line number	Diff line change
`@@ -182,7 +182,8 @@ class HttpTraceContext : public context::propagation::TextMapPropagator`
`182`	`182`
`183`	`183`	`static SpanContext ExtractImpl(const context::propagation::TextMapCarrier &carrier)`
`184`	`184`	`{`
`185`		`- nostd::string_view trace_parent = carrier.Get(kTraceParent);`
	`185`	`+ // Get trace_parent after trimming the leading and trailing whitespaces`
	`186`	`+ nostd::string_view trace_parent = detail::TrimString(carrier.Get(kTraceParent));`
`186`	`187`	`nostd::string_view trace_state = carrier.Get(kTraceState);`
`187`	`188`	`if (trace_parent == "")`
`188`	`189`	`{`