added e2e tests for status 413, 504

Author: lfortin

cgi-bin/timeout.js

@@ -0,0 +1,28 @@
+// Copyright (c) 2024-2025 lfortin
+//
+// Permission is hereby granted, free of charge, to any person obtaining
+// a copy of this software and associated documentation files (the
+// "Software"), to deal in the Software without restriction, including
+// without limitation the rights to use, copy, modify, merge, publish,
+// distribute, sublicense, and/or sell copies of the Software, and to
+// permit persons to whom the Software is furnished to do so, subject to
+// the following conditions:
+//
+// The above copyright notice and this permission notice shall be
+// included in all copies or substantial portions of the Software.
+//
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+// EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+// NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+// LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+// OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+// WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+
+process.stdout.write("Content-Type: text/plain\n");
+process.stdout.write("\n");
+
+// Simulate a long-running process
+setTimeout(() => {
+  process.stdout.write("Hello, World!");
+}, 5000);

cypress/e2e/spec.cy.js

@@ -40,6 +40,18 @@ describe("HTTP response status", () => {
       expect(response.body).to.equal("Not Found");
     });
   });
+  it("should get status 413 Entity Too Large", () => {
+    const bigPayload = "A".repeat(3 * 1024 * 1024); // 3 MB payload
+
+    cy.request({
+      url: "/cgi-bin/env.js",
+      method: "POST",
+      body: bigPayload,
+      failOnStatusCode: false,
+    }).then((response) => {
+      expect(response.status).to.equal(413);
+    });
+  });
   it("should get status 500 Internal Server Error", () => {
     cy.request({
       url: "/cgi-bin/error.js",
@@ -56,6 +68,14 @@ describe("HTTP response status", () => {
       expect(response.status).to.equal(200);
     });
   });
+  it("should get status 504 Gateway Timeout", () => {
+    cy.request({
+      url: "/cgi-bin/timeout.js",
+      failOnStatusCode: false,
+    }).then((response) => {
+      expect(response.status).to.equal(504);
+    });
+  });
 });
 describe("HTTP response body", () => {
   it("should have html payload", () => {

server/http.mjs

@@ -38,10 +38,10 @@ const callback = createHandler({
   indexExtension: "cgi",
   debugOutput: true,
   logRequests: true,
-  maxBuffer: 4 * 1024 ** 2,
+  maxBuffer: 1024 ** 2, // 1MB
   requestChunkSize: 4 * 1024,
   responseChunkSize: 4 * 1024,
-  requestTimeout: 30000,
+  requestTimeout: 3000, // 3s
   env: (env, req) => {
     return {
       REMOTE_AGENT: req.headers["user-agent"],

server/https.mjs

@@ -39,10 +39,10 @@ const callback = createHandler({
   indexExtension: "cgi",
   debugOutput: true,
   logRequests: true,
-  maxBuffer: 4 * 1024 ** 2,
+  maxBuffer: 1024 ** 2, // 1MB
   requestChunkSize: 4 * 1024,
   responseChunkSize: 4 * 1024,
-  requestTimeout: 30000,
+  requestTimeout: 3000, // 3s
   env: (env, req) => {
     return {
       REMOTE_AGENT: req.headers["user-agent"],