fix: dont modify immutable headers (#27)

nodejs · Sep 27, 2023 · ed01248 · ed01248
1 parent 6a94069
commit ed01248
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/handlers/get.ts b/src/handlers/get.ts
@@ -16,8 +16,6 @@ const getHandler: Handler = async (request, env, ctx, cache) => {
     const response = await cache.match(request);
 
     if (typeof response !== 'undefined') {
-      response.headers.append('x-cache-status', 'hit');
-
       return response;
     }
   }
@@ -51,7 +49,9 @@ const getHandler: Handler = async (request, env, ctx, cache) => {
 
   // Cache response if cache is enabled
   if (shouldServeCache && response.status !== 304 && response.status !== 206) {
-    ctx.waitUntil(cache.put(request, response.clone()));
+    const cached = response.clone();
+    cached.headers.append('x-cache-status', 'hit');
+    ctx.waitUntil(cache.put(request, cached));
   }
 
   response.headers.append('x-cache-status', 'miss');