@jaypie/express 1.2.4-rc11 → 1.2.4-rc13

package/dist/cjs/adapter/LambdaResponseBuffered.d.ts

@@ -30,6 +30,7 @@ export declare class LambdaResponseBuffered extends Writable {
     /**
      * Proxy for direct header access (e.g., res.headers['content-type']).
      * Required for compatibility with middleware like helmet that access headers directly.
+     * Uses direct _headers access to bypass dd-trace interception.
      */
     get headers(): Record<string, string | string[] | undefined>;
     writeHead(statusCode: number, statusMessageOrHeaders?: OutgoingHttpHeaders | string, headers?: OutgoingHttpHeaders): this;
@@ -52,6 +53,7 @@ export declare class LambdaResponseBuffered extends Writable {
     /**
      * Add a field to the Vary response header.
      * Used by CORS middleware to indicate response varies by Origin.
+     * Uses direct _headers access to bypass dd-trace interception.
      */
     vary(field: string): this;
     _write(chunk: Buffer | string, encoding: BufferEncoding, // eslint-disable-line no-undef

package/dist/cjs/adapter/LambdaResponseStreaming.d.ts

@@ -30,6 +30,7 @@ export declare class LambdaResponseStreaming extends Writable {
     /**
      * Proxy for direct header access (e.g., res.headers['content-type']).
      * Required for compatibility with middleware like helmet that access headers directly.
+     * Uses direct _headers access to bypass dd-trace interception.
      */
     get headers(): Record<string, string | string[] | undefined>;
     writeHead(statusCode: number, statusMessageOrHeaders?: OutgoingHttpHeaders | string, headers?: OutgoingHttpHeaders): this;
@@ -53,6 +54,7 @@ export declare class LambdaResponseStreaming extends Writable {
     /**
      * Add a field to the Vary response header.
      * Used by CORS middleware to indicate response varies by Origin.
+     * Uses direct _headers access to bypass dd-trace interception.
      */
     vary(field: string): this;
     _write(chunk: Buffer | string, encoding: BufferEncoding, // eslint-disable-line no-undef

package/dist/cjs/index.cjs

@@ -303,36 +303,40 @@ class LambdaResponseBuffered extends node_stream.Writable {
     /**
      * Proxy for direct header access (e.g., res.headers['content-type']).
      * Required for compatibility with middleware like helmet that access headers directly.
+     * Uses direct _headers access to bypass dd-trace interception.
      */
     get headers() {
         return new Proxy({}, {
             deleteProperty: (_target, prop) => {
-                this.removeHeader(String(prop));
+                this._headers.delete(String(prop).toLowerCase());
                 return true;
             },
             get: (_target, prop) => {
                 if (typeof prop === "symbol")
                     return undefined;
-                return this.getHeader(String(prop));
+                return this._headers.get(String(prop).toLowerCase());
             },
             getOwnPropertyDescriptor: (_target, prop) => {
-                if (this.hasHeader(String(prop))) {
+                const lowerProp = String(prop).toLowerCase();
+                if (this._headers.has(lowerProp)) {
                     return {
                         configurable: true,
                         enumerable: true,
-                        value: this.getHeader(String(prop)),
+                        value: this._headers.get(lowerProp),
                     };
                 }
                 return undefined;
             },
             has: (_target, prop) => {
-                return this.hasHeader(String(prop));
+                return this._headers.has(String(prop).toLowerCase());
             },
             ownKeys: () => {
-                return this.getHeaderNames();
+                return Array.from(this._headers.keys());
             },
             set: (_target, prop, value) => {
-                this.setHeader(String(prop), value);
+                if (!this._headersSent) {
+                    this._headers.set(String(prop).toLowerCase(), value);
+                }
                 return true;
             },
         });
@@ -349,9 +353,10 @@ class LambdaResponseBuffered extends node_stream.Writable {
             headersToSet = statusMessageOrHeaders;
         }
         if (headersToSet) {
+            // Use direct _headers access to bypass dd-trace interception
             for (const [key, value] of Object.entries(headersToSet)) {
                 if (value !== undefined) {
-                    this.setHeader(key, value);
+                    this._headers.set(key.toLowerCase(), String(value));
                 }
             }
         }
@@ -388,7 +393,8 @@ class LambdaResponseBuffered extends node_stream.Writable {
         return this;
     }
     json(data) {
-        this.setHeader("content-type", "application/json");
+        // Use direct _headers access to bypass dd-trace interception
+        this._headers.set("content-type", "application/json");
         this.end(JSON.stringify(data));
         return this;
     }
@@ -402,11 +408,12 @@ class LambdaResponseBuffered extends node_stream.Writable {
     /**
      * Add a field to the Vary response header.
      * Used by CORS middleware to indicate response varies by Origin.
+     * Uses direct _headers access to bypass dd-trace interception.
      */
     vary(field) {
-        const existing = this.getHeader("vary");
+        const existing = this._headers.get("vary");
         if (!existing) {
-            this.setHeader("vary", field);
+            this._headers.set("vary", field);
         }
         else {
             // Append to existing Vary header if field not already present
@@ -414,7 +421,7 @@ class LambdaResponseBuffered extends node_stream.Writable {
                 .split(",")
                 .map((f) => f.trim().toLowerCase());
             if (!fields.includes(field.toLowerCase())) {
-                this.setHeader("vary", `${existing}, ${field}`);
+                this._headers.set("vary", `${existing}, ${field}`);
             }
         }
         return this;
@@ -607,36 +614,42 @@ class LambdaResponseStreaming extends node_stream.Writable {
     /**
      * Proxy for direct header access (e.g., res.headers['content-type']).
      * Required for compatibility with middleware like helmet that access headers directly.
+     * Uses direct _headers access to bypass dd-trace interception.
      */
     get headers() {
         return new Proxy({}, {
             deleteProperty: (_target, prop) => {
-                this.removeHeader(String(prop));
+                if (!this._headersSent) {
+                    this._headers.delete(String(prop).toLowerCase());
+                }
                 return true;
             },
             get: (_target, prop) => {
                 if (typeof prop === "symbol")
                     return undefined;
-                return this.getHeader(String(prop));
+                return this._headers.get(String(prop).toLowerCase());
             },
             getOwnPropertyDescriptor: (_target, prop) => {
-                if (this.hasHeader(String(prop))) {
+                const lowerProp = String(prop).toLowerCase();
+                if (this._headers.has(lowerProp)) {
                     return {
                         configurable: true,
                         enumerable: true,
-                        value: this.getHeader(String(prop)),
+                        value: this._headers.get(lowerProp),
                     };
                 }
                 return undefined;
             },
             has: (_target, prop) => {
-                return this.hasHeader(String(prop));
+                return this._headers.has(String(prop).toLowerCase());
             },
             ownKeys: () => {
-                return this.getHeaderNames();
+                return Array.from(this._headers.keys());
             },
             set: (_target, prop, value) => {
-                this.setHeader(String(prop), value);
+                if (!this._headersSent) {
+                    this._headers.set(String(prop).toLowerCase(), value);
+                }
                 return true;
             },
         });
@@ -656,9 +669,10 @@ class LambdaResponseStreaming extends node_stream.Writable {
             headersToSet = statusMessageOrHeaders;
         }
         if (headersToSet) {
+            // Use direct _headers access to bypass dd-trace interception
             for (const [key, value] of Object.entries(headersToSet)) {
                 if (value !== undefined) {
-                    this.setHeader(key, value);
+                    this._headers.set(key.toLowerCase(), String(value));
                 }
             }
         }
@@ -716,7 +730,8 @@ class LambdaResponseStreaming extends node_stream.Writable {
         return this;
     }
     json(data) {
-        this.setHeader("content-type", "application/json");
+        // Use direct _headers access to bypass dd-trace interception
+        this._headers.set("content-type", "application/json");
         this.end(JSON.stringify(data));
         return this;
     }
@@ -730,11 +745,12 @@ class LambdaResponseStreaming extends node_stream.Writable {
     /**
      * Add a field to the Vary response header.
      * Used by CORS middleware to indicate response varies by Origin.
+     * Uses direct _headers access to bypass dd-trace interception.
      */
     vary(field) {
-        const existing = this.getHeader("vary");
+        const existing = this._headers.get("vary");
         if (!existing) {
-            this.setHeader("vary", field);
+            this._headers.set("vary", field);
         }
         else {
             // Append to existing Vary header if field not already present
@@ -742,7 +758,7 @@ class LambdaResponseStreaming extends node_stream.Writable {
                 .split(",")
                 .map((f) => f.trim().toLowerCase());
             if (!fields.includes(field.toLowerCase())) {
-                this.setHeader("vary", `${existing}, ${field}`);
+                this._headers.set("vary", `${existing}, ${field}`);
             }
         }
         return this;