stop cloning request body

Author: mistval

src/helpers/cache_keys.ts

@@ -55,15 +55,13 @@ function getBodyCacheKeyJson(body: unknown): string | object | undefined {
 }
 
 async function getRequestCacheKeyJson(request: Request) {
-  const body = await request.arrayBuffer();
-
   return {
     headers: getHeadersCacheKeyJson(Array.from(request.headers.entries())),
     method: request.method,
     redirect: request.redirect,
     referrer: request.referrer,
     url: request.url,
-    body: getBodyCacheKeyJson(body),
+    body: 'request_body' as undefined | string | object,
     // Confirmed that this property exists, but it's not in the types
     follow: (request as any).follow, // eslint-disable-line @typescript-eslint/no-unsafe-assignment
     // Confirmed that this property exists, but it's not in the types

src/index.ts

@@ -38,18 +38,12 @@ async function getResponse(
   resource: FetchResource,
   init: FetchInit,
 ) {
-  const originalResource = resource;
-
   if (typeof resource !== 'string' && !(resource instanceof Request)) {
     throw new TypeError(
       'The first argument to fetch must be either a string or a fetch Request instance',
     );
   }
 
-  if (originalResource instanceof Request) {
-    resource = originalResource.clone()
-  }
-
   const cacheKey = await fetchCustomization.calculateCacheKey(resource, init);
   const ejectSelfFromCache = async () => fetchCustomization.cache.remove(cacheKey);
 
@@ -79,10 +73,6 @@ async function getResponse(
         true,
       );
     }
-
-    if (originalResource instanceof Request) {
-      resource = originalResource.clone()
-    }
 
     const fetchResponse = await fetch(resource, init);
     const serializedMeta = NFCResponse.serializeMetaFromNodeFetchResponse(fetchResponse);