fix: do not crash in wrangler dev when passing a request object to fe…

…tch (#2597)

This reverts and fixes the changes in #1769
which does not support creating requests from requests whose bodies have already been consumed.

Fixes #2562

.changeset/perfect-grapes-fold.md

@@ -0,0 +1,10 @@
+---
+"wrangler": patch
+---
+
+fix: do not crash in wrangler dev when passing a request object to fetch
+
+This reverts and fixes the changes in https://github.com/cloudflare/wrangler2/pull/1769
+which does not support creating requests from requests whose bodies have already been consumed.
+
+Fixes #2562

fixtures/worker-app/src/index.js

@@ -1,13 +1,18 @@
 import { now } from "./dep";
 export default {
-	fetch(request) {
+	async fetch(request) {
 		console.log(
 			request.method,
 			request.url,
 			new Map([...request.headers]),
 			request.cf
 		);
 
+		await fetch(new URL("https://example.com"));
+		await fetch(
+			new Request("https://example.com", { method: "POST", body: "foo" })
+		);
+
 		return new Response(`${request.url} ${now()}`);
 	},
 

packages/wrangler/templates/checked-fetch.js

@@ -1,7 +1,15 @@
 const urls = new Set();
 
 export function checkedFetch(request, init) {
-	const url = new URL(new Request(request, init).url);
+	const url =
+		request instanceof URL
+			? request
+			: new URL(
+					(typeof request === "string"
+						? new Request(request, init)
+						: request
+					).url
+			  );
 	if (url.port && url.port !== "443" && url.protocol === "https:") {
 		if (!urls.has(url.toString())) {
 			urls.add(url.toString());