Fix multiple request logging (#8883)

* fix multiple request logging

* Create sour-islands-watch.md

Author: penalosa

.changeset/sour-islands-watch.md

@@ -0,0 +1,6 @@
+---
+"miniflare": patch
+"wrangler": patch
+---
+
+fix: Only log requests to the Wrangler dev server once

packages/miniflare/src/workers/core/entry.worker.ts

@@ -28,6 +28,7 @@ type Env = {
 	[CoreBindings.DURABLE_OBJECT_NAMESPACE_PROXY]: DurableObjectNamespace;
 	[CoreBindings.DATA_PROXY_SHARED_SECRET]?: ArrayBuffer;
 	[CoreBindings.TRIGGER_HANDLERS]: boolean;
+	[CoreBindings.LOG_REQUESTS]: boolean;
 } & {
 	[K in `${typeof CoreBindings.SERVICE_USER_ROUTE_PREFIX}${string}`]:
 		| Fetcher
@@ -421,7 +422,9 @@ export default <ExportedHandler<Env>>{
 			}
 			response = maybeInjectLiveReload(response, env, ctx);
 			response = ensureAcceptableEncoding(clientAcceptEncoding, response);
-			maybeLogRequest(request, response, env, ctx, startTime);
+			if (env[CoreBindings.LOG_REQUESTS]) {
+				maybeLogRequest(request, response, env, ctx, startTime);
+			}
 			return response;
 		} catch (e: any) {
 			return new Response(e?.stack ?? String(e), { status: 500 });

packages/wrangler/e2e/dev.test.ts

@@ -69,6 +69,9 @@ describe.each([{ cmd: "wrangler dev" }, { cmd: "wrangler dev --remote" }])(
 
 			await worker.waitForReload();
 
+			// Regression test for issue where multiple request logs were being logged per request
+			expect([...worker.currentOutput.matchAll(/GET /g)].length).toBe(1);
+
 			await expect(fetchText(url)).resolves.toMatchSnapshot();
 		});