fix(cloudflare-pages): remove .html extension from _routes.json (#…

…2498)
nitrojs · Jun 27, 2024 · 1b8a8ea · 1b8a8ea
1 parent 4e79242
commit 1b8a8ea
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/package.json b/package.json
@@ -185,4 +185,4 @@
       ]
     }
   }
-}
+}
diff --git a/src/presets/cloudflare-pages.ts b/src/presets/cloudflare-pages.ts
@@ -125,8 +125,16 @@ async function writeCFRoutes(nitro: Nitro) {
       ),
     ],
   });
-  routes.exclude.push(
-    ...publicAssetFiles.map((i) => withLeadingSlash(i)).sort(comparePaths)
+  // Remove index.html or the .html extension to support pages pre-rendering
+  routes.exclude!.push(
+    ...publicAssetFiles
+      .map(
+        (i) =>
+          withLeadingSlash(i)
+            .replace(/\/index\.html$/, "")
+            .replace(/\.html$/, "") || "/"
+      )
+      .sort(comparePaths)
   );
 
   // Only allow 100 rules in total (include + exclude)

diff --git a/test/presets/cloudflare-pages.test.ts b/test/presets/cloudflare-pages.test.ts
@@ -44,11 +44,11 @@ describe.skipIf(isWindows)("nitro:preset:cloudflare-pages", async () => {
           "/_unignored.txt",
           "/favicon.ico",
           "/json-string",
+          "/prerender",
           "/api/hello",
-          "/prerender/index.html",
+          "/api/hey",
           "/prerender/index.html.br",
           "/prerender/index.html.gz",
-          "/api/hey/index.html",
           "/api/param/foo.json",
           "/api/param/hidden",
           "/api/param/prerender1",