Additional test for baggage header and minor warn message improvement

Author: marco-saia-datadog

packages/core/src/rum/instrumentation/resourceTracking/requestProxy/XHRProxy/__tests__/baggageHeaderUtils.test.ts

@@ -103,4 +103,15 @@ describe('formatBaggageHeader', () => {
         const result = formatBaggageHeader(entries);
         expect(result).toBe('first=1,second=2,third=3');
     });
+
+    it('should trim keys and values', () => {
+        const entries = new Set([
+            'traceId=abc123;sampled=true;debug',
+            'test1  = this is a test'
+        ]);
+        const result = formatBaggageHeader(entries);
+        expect(result).toBe(
+            'traceId=abc123;sampled=true;debug,test1=this%20is%20a%20test'
+        );
+    });
 });

packages/core/src/rum/instrumentation/resourceTracking/requestProxy/XHRProxy/baggageHeaderUtils.ts

@@ -102,7 +102,7 @@ export function formatBaggageHeader(entries: Set<string>): string | null {
 
     if (formattedParts.length > MAX_MEMBERS) {
         InternalLog.log(
-            `XHRProxy: Too many baggage members: ${formattedParts.length} > ${MAX_MEMBERS} - entries might be dropped (https://www.w3.org/TR/baggage/#limits)`,
+            `XHRProxy: Too many baggage members: ${formattedParts.length} > ${MAX_MEMBERS} - entries may be dropped (https://www.w3.org/TR/baggage/#limits)`,
             SdkVerbosity.WARN
         );
     } else if (formattedParts.length === 0) {
@@ -114,7 +114,7 @@ export function formatBaggageHeader(entries: Set<string>): string | null {
 
     if (byteLength > MAX_BYTES) {
         InternalLog.log(
-            `Baggage header too large: ${byteLength} bytes > ${MAX_BYTES} - entries might be dropped (https://www.w3.org/TR/baggage/#limits)`,
+            `Baggage header too large: ${byteLength} bytes > ${MAX_BYTES} - entries may be dropped (https://www.w3.org/TR/baggage/#limits)`,
             SdkVerbosity.WARN
         );
     }