Fix TraceContextTest.test_traceparent_version_* tests

Author: psx95

api/include/opentelemetry/trace/propagation/http_trace_context.h

@@ -87,6 +87,7 @@ class HttpTraceContext : public context::propagation::TextMapPropagator
 
 private:
   static constexpr uint8_t kInvalidVersion = 0xFF;
+  static constexpr uint8_t kDefaultAssumedVersion = 0x00;
 
   static bool IsValidVersion(nostd::string_view version_hex)
   {
@@ -95,6 +96,13 @@ class HttpTraceContext : public context::propagation::TextMapPropagator
     return version != kInvalidVersion;
   }
 
+  static bool IsHigherVersion(nostd::string_view version_hex)
+  {
+    uint8_t version;
+    detail::HexToBinary(version_hex, &version, sizeof(version));
+    return version > kDefaultAssumedVersion;
+  }
+
   static void InjectImpl(context::propagation::TextMapCarrier &carrier,
                          const SpanContext &span_context)
   {
@@ -122,11 +130,6 @@ class HttpTraceContext : public context::propagation::TextMapPropagator
   static SpanContext ExtractContextFromTraceHeaders(nostd::string_view trace_parent,
                                                     nostd::string_view trace_state)
   {
-    if (trace_parent.size() != kTraceParentSize)
-    {
-      return SpanContext::GetInvalid();
-    }
-
     std::array<nostd::string_view, 4> fields{};
     if (detail::SplitString(trace_parent, '-', fields.data(), 4) != 4)
     {
@@ -155,6 +158,16 @@ class HttpTraceContext : public context::propagation::TextMapPropagator
       return SpanContext::GetInvalid();
     }
 
+    if (IsHigherVersion(version_hex) && trace_parent.size() < kTraceParentSize)
+    {
+      return SpanContext::GetInvalid();
+    }
+
+    if (!IsHigherVersion(version_hex) && trace_parent.size() != kTraceParentSize)
+    {
+      return SpanContext::GetInvalid();
+    }
+
     TraceId trace_id = TraceIdFromHex(trace_id_hex);
     SpanId span_id   = SpanIdFromHex(span_id_hex);