fix(cloudflare): allow spaces in filenames (#3047)

* Added URL decode to Cloudflare adapter static logic. * Added changeset. * Updated to attempt URL decode and ignore on fail. Co-authored-by: Luke Edwards <[email protected]> * Fixed formatting issues. Co-authored-by: Luke Edwards <[email protected]>
sveltejs · Dec 21, 2021 · ad321f8 · ad321f8
1 parent 5f4d078
commit ad321f8
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/.changeset/twenty-turkeys-promise.md b/.changeset/twenty-turkeys-promise.md
@@ -0,0 +1,5 @@
+---
+'@sveltejs/adapter-cloudflare': patch
+---
+
+Updated Cloudflare adapter to allow static files with spaces (eg. "Example File.pdf") to be accessed.
diff --git a/packages/adapter-cloudflare/files/worker.js b/packages/adapter-cloudflare/files/worker.js
@@ -6,8 +6,16 @@ init();
 export default {
 	async fetch(req, env) {
 		const url = new URL(req.url);
+
 		// check generated asset_set for static files
-		if (ASSETS.has(url.pathname.substring(1))) {
+		let pathname = url.pathname.substring(1);
+		try {
+			pathname = decodeURIComponent(pathname);
+		} catch (err) {
+			// ignore
+		}
+
+		if (ASSETS.has(pathname)) {
 			return env.ASSETS.fetch(req);
 		}