add tests for parsing headers

Author: brandon-pereira

packages/node-opentelemetry/src/__tests__/parseHeaders.ts

@@ -0,0 +1,71 @@
+import { parseOtlpHeaders } from '../otel';
+
+describe('Parse OTLP Headers', () => {
+  it('should return an empty object when no headers string is provided', () => {
+    expect(parseOtlpHeaders()).toEqual({});
+  });
+
+  it('should correctly parse a single header', () => {
+    expect(parseOtlpHeaders('key1=value1')).toEqual({ key1: 'value1' });
+  });
+
+  it('should correctly parse multiple headers', () => {
+    expect(parseOtlpHeaders('key1=value1,key2=value2')).toEqual({
+      key1: 'value1',
+      key2: 'value2',
+    });
+  });
+
+  it('should handle spaces around keys and values', () => {
+    expect(parseOtlpHeaders(' key1 = value1 , key2 = value2 ')).toEqual({
+      key1: 'value1',
+      key2: 'value2',
+    });
+  });
+
+  it('should ignore malformed headers without "="', () => {
+    expect(parseOtlpHeaders('key1=value1,malformedHeader,key2=value2')).toEqual(
+      {
+        key1: 'value1',
+        key2: 'value2',
+      },
+    );
+  });
+
+  it('should handle empty values', () => {
+    expect(parseOtlpHeaders('key1=,key2=value2')).toEqual({
+      key1: '',
+      key2: 'value2',
+    });
+  });
+
+  it('should handle special characters in values', () => {
+    expect(
+      parseOtlpHeaders(
+        'Authorization=Bearer token123,Content-Type=application/json',
+      ),
+    ).toEqual({
+      Authorization: 'Bearer token123',
+      'Content-Type': 'application/json',
+    });
+  });
+
+  it('should handle trailing comma', () => {
+    expect(parseOtlpHeaders('key1=value1,')).toEqual({
+      key1: 'value1',
+    });
+  });
+
+  it('should handle leading comma', () => {
+    expect(parseOtlpHeaders(',key1=value1')).toEqual({
+      key1: 'value1',
+    });
+  });
+
+  it('should handle multiple consecutive commas', () => {
+    expect(parseOtlpHeaders('key1=value1,,key2=value2')).toEqual({
+      key1: 'value1',
+      key2: 'value2',
+    });
+  });
+});

packages/node-opentelemetry/src/otel.ts

@@ -150,7 +150,9 @@ const hrtimeToMs = (hrtime: [number, number]) => {
  * Parses OTEL_EXPORTER_OTLP_HEADERS environment variable into a structured headers object.
  * Format: "key1=value1,key2=value2" -> { key1: "value1", key2: "value2" }
  */
-const parseOtlpHeaders = (headersString?: string): Record<string, string> => {
+export const parseOtlpHeaders = (
+  headersString?: string,
+): Record<string, string> => {
   if (!headersString) {
     return {};
   }