@cloudflare/pages-shared 0.3.0 → 0.3.1

package/asset-server/handler.ts

@@ -204,16 +204,7 @@ export async function generateHandler<
 			staticRedirectsMatcher() || generateRedirectsMatcher()({ request })[0];
 
 		if (match) {
-			if (match.status === 200) {
-				// A 200 redirect means that we are proxying to a different asset, for example,
-				// a request with url /users/12345 could be pointed to /users/id.html. In order to
-				// do this, we overwrite the pathname, and instead match for assets with that url,
-				// and importantly, do not use the regular redirect handler - as the url visible to
-				// the user does not change
-				pathname = new URL(match.to, request.url).pathname;
-			} else {
-				return getResponseFromMatch(match, url);
-			}
+			return getResponseFromMatch(match, url);
 		}
 
 		if (!request.method.match(/^(get|head)$/i)) {

package/metadata-generator/constants.ts

@@ -3,7 +3,7 @@ export const HEADERS_VERSION = 2;
 export const ANALYTICS_VERSION = 1;
 export const ROUTES_JSON_VERSION = 1;
 
-export const PERMITTED_STATUS_CODES = new Set([200, 301, 302, 303, 307, 308]);
+export const PERMITTED_STATUS_CODES = new Set([301, 302, 303, 307, 308]);
 export const HEADER_SEPARATOR = ":";
 export const MAX_LINE_LENGTH = 2000;
 export const MAX_HEADER_RULES = 100;

package/metadata-generator/parseRedirects.ts

@@ -104,7 +104,7 @@ export function parseRedirects(input: string): ParsedRedirects {
 			invalid.push({
 				line,
 				lineNumber: i + 1,
-				message: `Valid status codes are 200, 301, 302 (default), 303, 307, or 308. Got ${str_status}.`,
+				message: `Valid status codes are 301, 302 (default), 303, 307, or 308. Got ${str_status}.`,
 			});
 			continue;
 		}
@@ -119,22 +119,6 @@ export function parseRedirects(input: string): ParsedRedirects {
 		}
 		seen_paths.add(from);
 
-		if (status === 200) {
-			// Error can only be that it's not relative - given validateUrl is called above without onlyRelative:true,
-			// so if it's present, we can error to the user that proxying only expects relative urls
-			const [proxyTo, error] = validateUrl(to, true, true, true);
-			if (error) {
-				invalid.push({
-					line,
-					lineNumber: i + 1,
-					message: `Proxy (200) redirects can only point to relative paths. Got ${to}`,
-				});
-				continue;
-			}
-			rules.push({ from, to: proxyTo as string, status, lineNumber: i + 1 });
-			continue;
-		}
-
 		rules.push({ from, to, status, lineNumber: i + 1 });
 	}
 

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@cloudflare/pages-shared",
-	"version": "0.3.0",
+	"version": "0.3.1",
 	"repository": {
 		"type": "git",
 		"url": "https://github.com/cloudflare/workers-sdk.git",