WC-3583 Disable Sec-Fetch-Mode check when static routing is present

When the Router worker has static routing, the check against
"Sec-Fetch-Mode: navigate" is unnecessary. We have explicit static
routing to indicate whether or not we should go to a User worker or
the Asset worker, and should not try and guess via usually-set browser
headers

This adds a new parameter to unstable_canFetch RPC method, which should
be fine for backwards compatibility, and can be extended in the future
if needed. This was necessary because the Asset worker checks the
Request headers for Sec-Fetch-Mode to indicate if it can serve an asset
(including an index.html or 404.html page based on not_found_handling),
but static routing is only provided to the Router worker. Thus, we need
to pass more information over RPC

Author: matthewdavidrodgers

packages/workers-shared/asset-worker/src/configuration.ts

@@ -20,6 +20,7 @@ export const normalizeConfiguration = (
 			version: 2,
 			rules: {},
 		},
+		has_static_routing: configuration?.has_static_routing ?? false,
 		account_id: configuration?.account_id ?? -1,
 		script_id: configuration?.script_id ?? -1,
 		debug: configuration?.debug ?? false,

packages/workers-shared/asset-worker/src/handler.ts

@@ -203,7 +203,9 @@ export const canFetch = async (
 			flagIsEnabled(
 				configuration,
 				SEC_FETCH_MODE_NAVIGATE_HEADER_PREFERS_ASSET_SERVING
-			) && request.headers.get("Sec-Fetch-Mode") === "navigate"
+			) &&
+			request.headers.get("Sec-Fetch-Mode") === "navigate" &&
+			!configuration.has_static_routing
 		)
 	) {
 		configuration = {

packages/workers-shared/asset-worker/tests/handler.test.ts

@@ -1325,41 +1325,56 @@ describe("[Asset Worker] `canFetch`", () => {
 			[{ "Sec-Fetch-Mode": "cors" }, false],
 		] as const;
 
+		const staticRoutingModes = [
+			[false, true],
+			[true, false],
+		] as const;
+
 		const matrix = [];
 		for (const compatibilityOptions of compatibilityOptionsModes) {
 			for (const notFoundHandling of notFoundHandlingModes) {
 				for (const headers of headersModes) {
-					matrix.push({
-						compatibilityDate: compatibilityOptions[0].compatibilityDate,
-						compatibilityFlags: compatibilityOptions[0].compatibilityFlags,
-						notFoundHandling: notFoundHandling[0],
-						headers: headers[0],
-						expected:
-							compatibilityOptions[1] && notFoundHandling[1] && headers[1],
-					});
+					for (const hasStaticRouting of staticRoutingModes) {
+						matrix.push({
+							compatibilityDate: compatibilityOptions[0].compatibilityDate,
+							compatibilityFlags: compatibilityOptions[0].compatibilityFlags,
+							notFoundHandling: notFoundHandling[0],
+							headers: headers[0],
+							hasStaticRouting: hasStaticRouting[0],
+							expected:
+								compatibilityOptions[1] &&
+								notFoundHandling[1] &&
+								headers[1] &&
+								hasStaticRouting[1],
+						});
+					}
 				}
 			}
 		}
 
 		it.each(matrix)(
-			"compatibility_date $compatibilityDate, compatibility_flags $compatibilityFlags, not_found_handling $notFoundHandling, headers: $headers -> $expected",
+			"compatibility_date $compatibilityDate, compatibility_flags $compatibilityFlags, not_found_handling $notFoundHandling, headers: $headers, hasStaticRouting $hasStaticRouting -> $expected",
 			async ({
 				compatibilityDate,
 				compatibilityFlags,
 				notFoundHandling,
 				headers,
+				hasStaticRouting,
 				expected,
 			}) => {
 				expect(
 					await canFetch(
 						new Request("https://example.com/foo", { headers }),
 						// @ts-expect-error Empty config default to using mocked jaeger
 						mockEnv,
-						normalizeConfiguration({
-							compatibility_date: compatibilityDate,
-							compatibility_flags: compatibilityFlags,
-							not_found_handling: notFoundHandling,
-						}),
+						{
+							...normalizeConfiguration({
+								compatibility_date: compatibilityDate,
+								compatibility_flags: compatibilityFlags,
+								not_found_handling: notFoundHandling,
+								has_static_routing: hasStaticRouting,
+							}),
+						},
 						exists
 					)
 				).toBeTruthy();
@@ -1369,11 +1384,14 @@ describe("[Asset Worker] `canFetch`", () => {
 						new Request("https://example.com/bar", { headers }),
 						// @ts-expect-error Empty config default to using mocked jaeger
 						mockEnv,
-						normalizeConfiguration({
-							compatibility_date: compatibilityDate,
-							compatibility_flags: compatibilityFlags,
-							not_found_handling: notFoundHandling,
-						}),
+						{
+							...normalizeConfiguration({
+								compatibility_date: compatibilityDate,
+								compatibility_flags: compatibilityFlags,
+								not_found_handling: notFoundHandling,
+								has_static_routing: hasStaticRouting,
+							}),
+						},
 						exists
 					)
 				).toBe(expected);
@@ -1383,11 +1401,14 @@ describe("[Asset Worker] `canFetch`", () => {
 						new Request("https://example.com/", { headers }),
 						// @ts-expect-error Empty config default to using mocked jaeger
 						mockEnv,
-						normalizeConfiguration({
-							compatibility_date: compatibilityDate,
-							compatibility_flags: compatibilityFlags,
-							not_found_handling: notFoundHandling,
-						}),
+						{
+							...normalizeConfiguration({
+								compatibility_date: compatibilityDate,
+								compatibility_flags: compatibilityFlags,
+								not_found_handling: notFoundHandling,
+								has_static_routing: hasStaticRouting,
+							}),
+						},
 						exists
 					)
 				).toBeTruthy();
@@ -1397,11 +1418,14 @@ describe("[Asset Worker] `canFetch`", () => {
 						new Request("https://example.com/404", { headers }),
 						// @ts-expect-error Empty config default to using mocked jaeger
 						mockEnv,
-						normalizeConfiguration({
-							compatibility_date: compatibilityDate,
-							compatibility_flags: compatibilityFlags,
-							not_found_handling: notFoundHandling,
-						}),
+						{
+							...normalizeConfiguration({
+								compatibility_date: compatibilityDate,
+								compatibility_flags: compatibilityFlags,
+								not_found_handling: notFoundHandling,
+								has_static_routing: hasStaticRouting,
+							}),
+						},
 						exists
 					)
 				).toBeTruthy();

packages/workers-shared/utils/types.ts

@@ -74,6 +74,7 @@ export const AssetConfigSchema = z.object({
 		.optional(),
 	redirects: RedirectsSchema,
 	headers: HeadersSchema,
+	has_static_routing: z.boolean().optional(),
 	...InternalConfigSchema.shape,
 });