@jaypie/express 1.2.4-rc4 → 1.2.4-rc6

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

@@ -33,11 +33,13 @@ export declare class LambdaResponseBuffered extends Writable {
     /**
      * Express-style alias for getHeader().
      * Used by middleware like decorateResponse that use res.get().
+     * Note: Directly accesses _headers to avoid prototype chain issues with bundled code.
      */
     get(name: string): number | string | string[] | undefined;
     /**
      * Express-style alias for setHeader().
      * Used by middleware like decorateResponse that use res.set().
+     * Note: Directly accesses _headers to avoid prototype chain issues with bundled code.
      */
     set(name: string, value: number | string | string[]): this;
     status(code: number): this;

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

@@ -34,11 +34,13 @@ export declare class LambdaResponseStreaming extends Writable {
     /**
      * Express-style alias for getHeader().
      * Used by middleware like decorateResponse that use res.get().
+     * Note: Directly accesses _headers to avoid prototype chain issues with bundled code.
      */
     get(name: string): number | string | string[] | undefined;
     /**
      * Express-style alias for setHeader().
      * Used by middleware like decorateResponse that use res.set().
+     * Note: Directly accesses _headers to avoid prototype chain issues with bundled code.
      */
     set(name: string, value: number | string | string[]): this;
     status(code: number): this;

package/dist/cjs/index.cjs

@@ -1,6 +1,7 @@
 'use strict';
 
 var node_stream = require('node:stream');
+var node_http = require('node:http');
 var errors = require('@jaypie/errors');
 var kit = require('@jaypie/kit');
 var expressCors = require('cors');
@@ -161,6 +162,10 @@ function createLambdaRequest(event, context) {
 //
 // Constants
 //
+// Get Node's internal kOutHeaders symbol from ServerResponse prototype.
+// This is needed for compatibility with Datadog dd-trace instrumentation,
+// which patches HTTP methods and expects this internal state to exist.
+const kOutHeaders$1 = Object.getOwnPropertySymbols(node_http.ServerResponse.prototype).find((s) => s.toString() === "Symbol(kOutHeaders)");
 const BINARY_CONTENT_TYPE_PATTERNS = [
     /^application\/octet-stream$/,
     /^application\/pdf$/,
@@ -191,6 +196,11 @@ class LambdaResponseBuffered extends node_stream.Writable {
         this._headers = new Map();
         this._headersSent = false;
         this._resolve = null;
+        // Initialize Node's internal kOutHeaders for dd-trace compatibility.
+        // dd-trace patches HTTP methods and expects this internal state.
+        if (kOutHeaders$1) {
+            this[kOutHeaders$1] = Object.create(null);
+        }
     }
     //
     // Promise-based API for getting final result
@@ -213,14 +223,31 @@ class LambdaResponseBuffered extends node_stream.Writable {
             // In production, log warning but don't throw to match Express behavior
             return this;
         }
-        this._headers.set(name.toLowerCase(), String(value));
+        const lowerName = name.toLowerCase();
+        this._headers.set(lowerName, String(value));
+        // Sync with kOutHeaders for dd-trace compatibility
+        // Node stores as { 'header-name': ['Header-Name', value] }
+        if (kOutHeaders$1) {
+            const outHeaders = this[kOutHeaders$1];
+            if (outHeaders) {
+                outHeaders[lowerName] = [name, String(value)];
+            }
+        }
         return this;
     }
     getHeader(name) {
         return this._headers.get(name.toLowerCase());
     }
     removeHeader(name) {
-        this._headers.delete(name.toLowerCase());
+        const lowerName = name.toLowerCase();
+        this._headers.delete(lowerName);
+        // Sync with kOutHeaders for dd-trace compatibility
+        if (kOutHeaders$1) {
+            const outHeaders = this[kOutHeaders$1];
+            if (outHeaders) {
+                delete outHeaders[lowerName];
+            }
+        }
     }
     getHeaders() {
         const headers = {};
@@ -302,16 +329,21 @@ class LambdaResponseBuffered extends node_stream.Writable {
     /**
      * Express-style alias for getHeader().
      * Used by middleware like decorateResponse that use res.get().
+     * Note: Directly accesses _headers to avoid prototype chain issues with bundled code.
      */
     get(name) {
-        return this.getHeader(name);
+        return this._headers.get(name.toLowerCase());
     }
     /**
      * Express-style alias for setHeader().
      * Used by middleware like decorateResponse that use res.set().
+     * Note: Directly accesses _headers to avoid prototype chain issues with bundled code.
      */
     set(name, value) {
-        return this.setHeader(name, value);
+        if (!this._headersSent) {
+            this._headers.set(name.toLowerCase(), String(value));
+        }
+        return this;
     }
     status(code) {
         this.statusCode = code;
@@ -409,6 +441,14 @@ class LambdaResponseBuffered extends node_stream.Writable {
     }
 }
 
+//
+//
+// Constants
+//
+// Get Node's internal kOutHeaders symbol from ServerResponse prototype.
+// This is needed for compatibility with Datadog dd-trace instrumentation,
+// which patches HTTP methods and expects this internal state to exist.
+const kOutHeaders = Object.getOwnPropertySymbols(node_http.ServerResponse.prototype).find((s) => s.toString() === "Symbol(kOutHeaders)");
 //
 //
 // LambdaResponseStreaming Class
@@ -431,6 +471,11 @@ class LambdaResponseStreaming extends node_stream.Writable {
         this._pendingWrites = [];
         this._wrappedStream = null;
         this._responseStream = responseStream;
+        // Initialize Node's internal kOutHeaders for dd-trace compatibility.
+        // dd-trace patches HTTP methods and expects this internal state.
+        if (kOutHeaders) {
+            this[kOutHeaders] = Object.create(null);
+        }
     }
     //
     // Header management
@@ -441,7 +486,16 @@ class LambdaResponseStreaming extends node_stream.Writable {
             // Headers cannot be changed after body starts
             return this;
         }
-        this._headers.set(name.toLowerCase(), String(value));
+        const lowerName = name.toLowerCase();
+        this._headers.set(lowerName, String(value));
+        // Sync with kOutHeaders for dd-trace compatibility
+        // Node stores as { 'header-name': ['Header-Name', value] }
+        if (kOutHeaders) {
+            const outHeaders = this[kOutHeaders];
+            if (outHeaders) {
+                outHeaders[lowerName] = [name, String(value)];
+            }
+        }
         return this;
     }
     getHeader(name) {
@@ -449,7 +503,15 @@ class LambdaResponseStreaming extends node_stream.Writable {
     }
     removeHeader(name) {
         if (!this._headersSent) {
-            this._headers.delete(name.toLowerCase());
+            const lowerName = name.toLowerCase();
+            this._headers.delete(lowerName);
+            // Sync with kOutHeaders for dd-trace compatibility
+            if (kOutHeaders) {
+                const outHeaders = this[kOutHeaders];
+                if (outHeaders) {
+                    delete outHeaders[lowerName];
+                }
+            }
         }
     }
     getHeaders() {
@@ -556,16 +618,21 @@ class LambdaResponseStreaming extends node_stream.Writable {
     /**
      * Express-style alias for getHeader().
      * Used by middleware like decorateResponse that use res.get().
+     * Note: Directly accesses _headers to avoid prototype chain issues with bundled code.
      */
     get(name) {
-        return this.getHeader(name);
+        return this._headers.get(name.toLowerCase());
     }
     /**
      * Express-style alias for setHeader().
      * Used by middleware like decorateResponse that use res.set().
+     * Note: Directly accesses _headers to avoid prototype chain issues with bundled code.
      */
     set(name, value) {
-        return this.setHeader(name, value);
+        if (!this._headersSent) {
+            this._headers.set(name.toLowerCase(), String(value));
+        }
+        return this;
     }
     status(code) {
         this.statusCode = code;