fix: support non-lower-cased headers in LumigoW3CTraceContextPropagat…

…or (#162)
lumigo-io · Mar 4, 2023 · 3684534 · 3684534
1 parent 5a50be7
commit 3684534
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/propagator/w3cTraceContextPropagator.ts b/src/propagator/w3cTraceContextPropagator.ts
@@ -15,7 +15,11 @@ export class LumigoW3CTraceContextPropagator extends W3CTraceContextPropagator {
   override inject(context: Context, carrier: unknown, setter: TextMapSetter): void {
     if (typeof carrier === 'object') {
       for (const key of contextKeysSkipInject) {
-        if (key in carrier) {
+        if (
+          Object.keys(carrier)
+            .map((key) => key.toLowerCase())
+            .includes(key)
+        ) {
           diag.debug(`Skipping injection of trace context due to key '${key}' in carrier`);
           return;
         }

diff --git a/test/component/http/app/app.js b/test/component/http/app/app.js
@@ -72,7 +72,7 @@ const requestListener = async function (req, res) {
                 // We expect this to throw due to the timeout + impossibility to connect
                 await axios.get(`https://httpbin.org/status/201`, {
                     headers: {
-                        'x-amz-content-sha256': 'abcdefghi',
+                        'x-aMz-cOntEnt-sHa256': 'abcdefghi', // Creative upper-casing, but only marginally less insane than Amazon's :-)
                     },
                     timeout: 5_000, // Milliseconds
                 });