fix(nextjs-component, e2e-tests): allow all CloudFront HTTP methods for default caching behavior

packages/e2e-tests/next-app/cypress/integration/pages.test.ts

@@ -13,6 +13,16 @@ describe("Pages Tests", () => {
 
         cy.visit(path);
       });
+
+      ["HEAD", "DELETE", "POST", "GET", "OPTIONS", "PUT", "PATCH"].forEach(
+        (method) => {
+          it(`allows HTTP method: ${method}`, () => {
+            cy.request({ url: path, method: method }).then((response) => {
+              expect(response.status).to.equal(200);
+            });
+          });
+        }
+      );
     });
   });
 
@@ -31,6 +41,16 @@ describe("Pages Tests", () => {
 
         cy.visit(path);
       });
+
+      ["HEAD", "DELETE", "POST", "GET", "OPTIONS", "PUT", "PATCH"].forEach(
+        (method) => {
+          it(`allows HTTP method: ${method}`, () => {
+            cy.request({ url: path, method: method }).then((response) => {
+              expect(response.status).to.equal(200);
+            });
+          });
+        }
+      );
     });
   });
 
@@ -43,6 +63,27 @@ describe("Pages Tests", () => {
         cy.ensureRouteCached(path);
         cy.visit(path);
       });
+
+      ["HEAD", "GET"].forEach((method) => {
+        it(`allows HTTP method: ${method}`, () => {
+          cy.request({ url: path, method: method }).then((response) => {
+            expect(response.status).to.equal(200);
+          });
+        });
+      });
+
+      ["DELETE", "POST", "OPTIONS", "PUT", "PATCH"].forEach((method) => {
+        it(`disallows HTTP method with 4xx error: ${method}`, () => {
+          cy.request({
+            url: path,
+            method: method,
+            failOnStatusCode: false
+          }).then((response) => {
+            expect(response.status).to.be.at.least(400);
+            expect(response.status).to.be.lessThan(500);
+          });
+        });
+      });
     });
   });
 

packages/e2e-tests/next-app/cypress/integration/caching.test.ts → packages/e2e-tests/next-app/cypress/integration/static-files.test.ts

@@ -1,4 +1,4 @@
-describe("Caching Tests", () => {
+describe("Static Files Tests", () => {
   before(() => {
     cy.ensureAllRoutesNotErrored();
   });
@@ -17,7 +17,7 @@ describe("Caching Tests", () => {
     });
   });
 
-  describe("public files are cached", () => {
+  describe("public files", () => {
     [{ path: "/app-store-badge.png" }].forEach(({ path }) => {
       it(`serves and caches file ${path}`, () => {
         // Request once to ensure cached
@@ -27,6 +27,27 @@ describe("Caching Tests", () => {
           expect(response.headers["x-cache"]).to.equal("Hit from cloudfront");
         });
       });
+
+      ["HEAD", "GET"].forEach((method) => {
+        it(`allows HTTP method: ${method}`, () => {
+          cy.request({ url: path, method: method }).then((response) => {
+            expect(response.status).to.equal(200);
+          });
+        });
+      });
+
+      ["DELETE", "POST", "OPTIONS", "PUT", "PATCH"].forEach((method) => {
+        it(`disallows HTTP method with 4xx error: ${method}`, () => {
+          cy.request({
+            url: path,
+            method: method,
+            failOnStatusCode: false
+          }).then((response) => {
+            expect(response.status).to.be.at.least(400);
+            expect(response.status).to.be.lessThan(500);
+          });
+        });
+      });
     });
   });
 });

packages/serverless-components/nextjs-component/__tests__/custom-inputs.test.ts

@@ -684,7 +684,15 @@ describe("Custom inputs", () => {
         minTTL: 0,
         defaultTTL: 0,
         maxTTL: 31536000,
-        allowedHttpMethods: ["HEAD", "GET"],
+        allowedHttpMethods: [
+          "HEAD",
+          "DELETE",
+          "POST",
+          "GET",
+          "OPTIONS",
+          "PUT",
+          "PATCH"
+        ],
         forward: {
           cookies: "all",
           queryString: true

packages/serverless-components/nextjs-component/src/component.ts

@@ -520,7 +520,15 @@ class NextjsComponent extends Component {
           ...cloudFrontDefaults.forward
         },
         // everything after here cant be overridden
-        allowedHttpMethods: ["HEAD", "GET"],
+        allowedHttpMethods: [
+          "HEAD",
+          "DELETE",
+          "POST",
+          "GET",
+          "OPTIONS",
+          "PUT",
+          "PATCH"
+        ],
         "lambda@edge": {
           ...defaultLambdaAtEdgeConfig,
           "origin-request": `${defaultEdgeLambdaOutputs.arn}:${defaultEdgeLambdaPublishOutputs.version}`,