socket-function 1.2.26 → 1.2.27

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "socket-function",
-	"version": "1.2.26",
+	"version": "1.2.27",
 	"main": "index.js",
 	"license": "MIT",
 	"dependencies": {

package/src/callHTTPHandler.ts

@@ -13,6 +13,9 @@ import { createSingleton } from "./createSingleton";
 //  the HTTP handler of any other copy. See createSingleton.
 const defaultHTTPCall = createSingleton("callHTTPHandler.defaultHTTPCall", 1, () => ({ call: undefined as CallType | undefined }));
 
+// Statuses where the spec forbids a message body, so we must not write resultBuffer
+const BODYLESS_STATUS_CODES = new Set([204, 205, 304]);
+
 export function setDefaultHTTPCall(call: CallType) {
     defaultHTTPCall.get().call = call;
 }
@@ -253,12 +256,18 @@ export async function httpCallHandler(request: http.IncomingMessage, response: h
 
         if (headers) {
             for (let headerName in headers) {
+                // "status" is the status line, not a header, so don't emit it on the wire
+                if (headerName.toLowerCase() === "status") continue;
                 response.setHeader(headerName, headers[headerName]);
             }
-            let status = headers["status"];
+            let status = headers["status"] ?? headers["Status"];
             if (status) {
-                response.writeHead(+status);
-                return;
+                // Only set statusCode (NOT writeHead), as writeHead locks the header block, which would
+                //  drop the Content-Type / Content-Length / compression handling below
+                response.statusCode = +status;
+                if (BODYLESS_STATUS_CODES.has(+status) || +status < 200) {
+                    return;
+                }
             }
         }
         // With nosniff set, an untyped response is unusable to the browser, so results without explicit headers need their actual type (JSON) declared