Merge pull request #54 from vyuldashev/master

Headers should not be case sensitive.
jukylin · Aug 8, 2019 · 5598896 · 5598896
2 parents ba8cabf + 17ae462
commit 5598896
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/Jaeger/Propagator/JaegerPropagator.php b/src/Jaeger/Propagator/JaegerPropagator.php
@@ -37,12 +37,14 @@ public function inject(SpanContext $spanContext, $format, &$carrier){
      */
     public function extract($format, $carrier){
         $spanContext = new SpanContext(0, 0, 0, null, 0);
+
+        $carrier = array_change_key_case($carrier, CASE_LOWER);
+
         foreach ($carrier as $k => $v){
 
-            $k = strtolower($k);
             $v = urldecode($v);
             if($k == Constants\Tracer_State_Header_Name){
-                list($traceId, $spanId, $parentId,$flags) = explode(':', $carrier[strtoupper($k)]);
+                list($traceId, $spanId, $parentId,$flags) = explode(':', $carrier[$k]);
 
                 $spanContext->spanId = $spanContext->hexToSignedInt($spanId);
                 $spanContext->parentId = $spanContext->hexToSignedInt($parentId);
@@ -79,4 +81,4 @@ public function extract($format, $carrier){
         return $spanContext;
     }
 
-}
+}