From 8f3c150c695b92e9d2b76e1c1ba55aac47f6e59d Mon Sep 17 00:00:00 2001
From: Ariel Valentin <arielvalentin@users.noreply.github.com>
Date: Mon, 31 May 2021 07:55:54 -0500
Subject: [PATCH] fix: Support Case Insensitive Trace and Span IDs (#781)

This change allows systems that generate uppercase hex values to properly propagate OTTrace context.
---
 .../propagator/ottrace/text_map_propagator.rb |  4 ++--
 .../ottrace/text_map_propagator_test.rb       | 20 +++++++++++++++++++
 2 files changed, 22 insertions(+), 2 deletions(-)

diff --git a/propagator/ottrace/lib/opentelemetry/propagator/ottrace/text_map_propagator.rb b/propagator/ottrace/lib/opentelemetry/propagator/ottrace/text_map_propagator.rb
index 4a957e56d5..ad4682d323 100644
--- a/propagator/ottrace/lib/opentelemetry/propagator/ottrace/text_map_propagator.rb
+++ b/propagator/ottrace/lib/opentelemetry/propagator/ottrace/text_map_propagator.rb
@@ -18,8 +18,8 @@ module OTTrace
       # Propagates context using OTTrace header format
       class TextMapPropagator
         PADDING = '0' * 16
-        VALID_TRACE_ID_REGEX = /^[0-9a-f]{32}$/.freeze
-        VALID_SPAN_ID_REGEX = /^[0-9a-f]{16}$/.freeze
+        VALID_TRACE_ID_REGEX = /^[0-9a-f]{32}$/i.freeze
+        VALID_SPAN_ID_REGEX = /^[0-9a-f]{16}$/i.freeze
         TRACE_ID_64_BIT_WIDTH = 64 / 4
         TRACE_ID_HEADER = 'ot-tracer-traceid'
         SPAN_ID_HEADER = 'ot-tracer-spanid'
diff --git a/propagator/ottrace/test/opentelemetry/propagator/ottrace/text_map_propagator_test.rb b/propagator/ottrace/test/opentelemetry/propagator/ottrace/text_map_propagator_test.rb
index 8c20c583ea..a957a3879c 100644
--- a/propagator/ottrace/test/opentelemetry/propagator/ottrace/text_map_propagator_test.rb
+++ b/propagator/ottrace/test/opentelemetry/propagator/ottrace/text_map_propagator_test.rb
@@ -117,6 +117,26 @@ def set(carrier, key, value)
       end
     end
 
+    describe 'given a minimal context with uppercase fields' do
+      let(:carrier) do
+        {
+          'ot-tracer-traceid' => trace_id_header.upcase,
+          'ot-tracer-spanid' => span_id_header.upcase,
+          'ot-tracer-sampled' => sampled_header
+        }
+      end
+
+      it 'extracts parent context' do
+        context = propagator.extract(carrier, context: parent_context)
+        extracted_context = OpenTelemetry::Trace.current_span(context).context
+
+        _(extracted_context.hex_trace_id).must_equal('80f198ee56343ba864fe8b2a57d3eff7')
+        _(extracted_context.hex_span_id).must_equal('e457b5a2e4d86bd1')
+        _(extracted_context.trace_flags).must_equal(OpenTelemetry::Trace::TraceFlags::SAMPLED)
+        _(extracted_context).must_be(:remote?)
+      end
+    end
+
     describe 'given a context with sampling disabled' do
       let(:sampled_header) do
         'false'