@jaypie/express 1.2.4-rc12 → 1.2.4-rc14

package/dist/esm/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;

package/dist/esm/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;

package/dist/esm/index.js

@@ -301,36 +301,40 @@ 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() {
         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;
             },
         });
@@ -347,9 +351,10 @@ class LambdaResponseBuffered extends 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));
                 }
             }
         }
@@ -386,7 +391,8 @@ class LambdaResponseBuffered extends 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;
     }
@@ -606,36 +612,42 @@ 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() {
         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;
             },
         });
@@ -655,9 +667,10 @@ class LambdaResponseStreaming extends 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));
                 }
             }
         }
@@ -715,7 +728,8 @@ class LambdaResponseStreaming extends 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;
     }
@@ -1704,6 +1718,10 @@ function expressHandler(handlerOrOptions, optionsOrHandler) {
         catch (error) {
             log.fatal("Express encountered an error while sending the response");
             log.var({ responseError: error });
+            // Log full stack trace for debugging
+            if (error instanceof Error && error.stack) {
+                log.error(error.stack);
+            }
         }
         // Log response
         const extras = {};