Remove await from user Worker fetch in router-worker (#7410)

cloudflare · Dec 2, 2024 · 6b21919 · 6b21919
1 parent 515174e
commit 6b21919
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/.changeset/gorgeous-paws-compare.md b/.changeset/gorgeous-paws-compare.md
@@ -0,0 +1,5 @@
+---
+"@cloudflare/workers-shared": patch
+---
+
+chore: remove awaits from Asset Worker fetches. This has no user-facing impact.
diff --git a/packages/workers-shared/router-worker/src/index.ts b/packages/workers-shared/router-worker/src/index.ts
@@ -72,15 +72,15 @@ export default {
 			if (env.CONFIG.has_user_worker) {
 				if (await env.ASSET_WORKER.unstable_canFetch(request)) {
 					analytics.setData({ dispatchtype: DISPATCH_TYPE.ASSETS });
-					return await env.ASSET_WORKER.fetch(maybeSecondRequest);
+					return env.ASSET_WORKER.fetch(maybeSecondRequest);
 				} else {
 					analytics.setData({ dispatchtype: DISPATCH_TYPE.WORKER });
 					return env.USER_WORKER.fetch(maybeSecondRequest);
 				}
 			}
 
 			analytics.setData({ dispatchtype: DISPATCH_TYPE.ASSETS });
-			return await env.ASSET_WORKER.fetch(request);
+			return env.ASSET_WORKER.fetch(request);
 		} catch (err) {
 			if (err instanceof Error) {
 				analytics.setData({ error: err.message });