chore: copy body without using it

packages/astro/src/core/routing/request.ts

@@ -5,8 +5,6 @@
  * @param oldRequest The old `Request`
  */
 export async function copyRequest(newUrl: URL, oldRequest: Request): Promise<Request> {
-	const body = oldRequest.headers.get('content-type') ? oldRequest.blob() : Promise.resolve(undefined);
-	return body.then((requestBody) => {
 		return new Request(newUrl, {
 			method: oldRequest.method,
 			headers: oldRequest.headers,
@@ -20,8 +18,8 @@ export async function copyRequest(newUrl: URL, oldRequest: Request): Promise<Req
 			integrity: oldRequest.integrity,
 			signal: oldRequest.signal,
 			keepalive: oldRequest.keepalive,
+			// https://fetch.spec.whatwg.org/#dom-request-duplex
+			// @ts-expect-error It isn't part of the types, but undici accepts it and it allows to carry over the body to a new request
+			duplex: "half"
 		})
-	}
-
-		);
 }

packages/astro/test/fixtures/reroute/src/pages/post/post-a.astro

@@ -1,11 +1,5 @@
 ---
 let email = ''
-if (Astro.request.method === 'POST') {
-	try {
-		const data = await Astro.request.formData();
-		email = data.get('email')?.toString().trim();
-	} catch {}
-}
 return Astro.rewrite('/post/post-b')
 ---
 

packages/astro/test/fixtures/reroute/src/pages/post/post-b.astro

@@ -2,9 +2,11 @@
 let email = ''
 if (Astro.request.method === 'POST') {
 	try {
-		const data = await Astro.request.formData();
-		email = data.get('email')?.toString().trim();
-	} catch {}
+		const data = await Astro.request.json();
+		email = data.email?.toString().trim();
+	} catch (e) {
+		console.log(e)
+	}
 }
 ---
 

packages/astro/test/rewrite.test.js

@@ -190,17 +190,17 @@ describe('SSR reroute', () => {
 	});
 
 	it('should pass the POST data from one page to another', async () => {
-		const html = await fixture.fetch('/post/post-a', {
+		const request = new Request('http://example.com/post/post-a', {
 			method: "POST",
 			body: JSON.stringify({
 				email: "[email protected]",
-				title: "Fix my bugs",
-				completed: false
 			}),
 			headers: {
-				"Content-type": "application/json; charset=UTF-8"
+				"content-type": "application/json"
 			}
-		}).then((res) => res.text());
+		});
+		const response = await app.render(request);
+		const html = await response.text();
 		const $ = cheerioLoad(html);
 
 		assert.equal($('h1').text(), 'Post B');