@fastify/compress 9.1.0 → 9.1.1

package/.github/dependabot.yml

@@ -34,6 +34,7 @@ updates:
       - dependency-name: "eslint"
       - dependency-name: "neostandard"
       - dependency-name: "@stylistic/*"
+      - dependency-name: "typescript"
     open-pull-requests-limit: 10
     groups:
       # Production dependencies with breaking changes

package/.github/workflows/ci.yml

@@ -27,7 +27,7 @@ jobs:
     permissions:
       contents: write
       pull-requests: write
-    uses: fastify/workflows/.github/workflows/plugins-ci.yml@v6
+    uses: fastify/workflows/.github/workflows/plugins-ci.yml@2073dc8e1f9e172bf42daa3843c9dbd31af1e8cb # v6.0.0
     with:
       license-check: true
       lint: true

package/.github/workflows/lock-threads.yml

@@ -16,4 +16,4 @@ jobs:
     permissions:
       issues: write
       pull-requests: write
-    uses: fastify/workflows/.github/workflows/lock-threads.yml@v6
+    uses: fastify/workflows/.github/workflows/lock-threads.yml@2073dc8e1f9e172bf42daa3843c9dbd31af1e8cb # v6.0.0

package/index.js

@@ -18,6 +18,7 @@ const {
   isWebReadableStream,
   isFetchResponse,
   webStreamToNodeReadable,
+  unwrapFetchResponse,
   createPeekTransform
 } = require('./lib/utils')
 
@@ -296,7 +297,7 @@ function buildRouteCompress (_fastify, params, routeOptions, decorateOnly) {
 
     if (typeof payload.pipe !== 'function') {
       if (isFetchResponse(payload)) {
-        payload = payload.body
+        payload = unwrapFetchResponse(reply, payload)
       }
 
       if (isWebReadableStream(payload)) {
@@ -425,7 +426,7 @@ function compress (params) {
 
     if (typeof payload.pipe !== 'function') {
       if (isFetchResponse(payload)) {
-        payload = payload.body
+        payload = unwrapFetchResponse(this, payload)
       }
 
       if (isWebReadableStream(payload)) {

package/lib/utils.js

@@ -64,6 +64,14 @@ function webStreamToNodeReadable (webStream) {
   return NodeReadable.fromWeb(webStream)
 }
 
+function unwrapFetchResponse (reply, response) {
+  reply.code(response.status)
+  for (const [name, value] of response.headers) {
+    reply.header(name, value)
+  }
+  return response.body
+}
+
 /**
  * Provide a async iteratable for Readable.from
  */
@@ -192,5 +200,6 @@ module.exports = {
   isWebReadableStream,
   isFetchResponse,
   webStreamToNodeReadable,
+  unwrapFetchResponse,
   createPeekTransform
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fastify/compress",
-  "version": "9.1.0",
+  "version": "9.1.1",
   "description": "Fastify compression utils",
   "main": "index.js",
   "type": "commonjs",
@@ -14,8 +14,8 @@
   },
   "devDependencies": {
     "@types/node": "^26.0.1",
-    "adm-zip": "^0.5.12",
-    "c8": "^11.0.0",
+    "adm-zip": "^0.6.0",
+    "c8": "^12.0.0",
     "eslint": "^9.17.0",
     "fastify": "^5.0.0",
     "jsonstream": "^1.0.3",

package/test/global-compress.test.js

@@ -288,6 +288,36 @@ describe('When `global` is not set, it is `true` by default :', async () => {
     t.assert.equal(payload.toString('utf-8'), body)
   })
 
+  test('it should preserve a Fetch API Response status and headers', async (t) => {
+    t.plan(5)
+
+    const fastify = Fastify()
+    await fastify.register(compressPlugin, { threshold: 0 })
+
+    const body = 'hello from fetch response'
+    fastify.get('/fetch-resp', (_request, reply) => {
+      reply.send(new Response(body, {
+        status: 201,
+        headers: {
+          'content-type': 'text/plain',
+          'cache-control': 'public, max-age=120',
+          'set-cookie': 'a=b; HttpOnly'
+        }
+      }))
+    })
+
+    const response = await fastify.inject({
+      url: '/fetch-resp',
+      method: 'GET',
+      headers: { 'accept-encoding': 'gzip' }
+    })
+    t.assert.equal(response.statusCode, 201)
+    t.assert.equal(response.headers['content-type'], 'text/plain')
+    t.assert.equal(response.headers['cache-control'], 'public, max-age=120')
+    t.assert.equal(response.headers['set-cookie'], 'a=b; HttpOnly')
+    t.assert.equal(zlib.gunzipSync(response.rawPayload).toString('utf-8'), body)
+  })
+
   test('it should compress a Web ReadableStream body', async (t) => {
     t.plan(1)
 

package/test/routes-compress.test.js

@@ -364,6 +364,23 @@ test('reply.compress should handle Fetch Response', async (t) => {
   t.assert.equal(zlib.gunzipSync(res.rawPayload).toString('utf8'), 'from reply.compress')
 })
 
+test('reply.compress should preserve a Fetch Response status and headers', async (t) => {
+  t.plan(4)
+  const fastify = Fastify()
+  await fastify.register(compressPlugin, { global: true, threshold: 0 })
+  fastify.get('/', (_req, reply) => {
+    reply.compress(new Response('from reply.compress', {
+      status: 418,
+      headers: { 'content-type': 'text/plain', 'x-custom': 'kept' }
+    }))
+  })
+  const res = await fastify.inject({ url: '/', method: 'GET', headers: { 'accept-encoding': 'gzip' } })
+  t.assert.equal(res.statusCode, 418)
+  t.assert.equal(res.headers['content-type'], 'text/plain')
+  t.assert.equal(res.headers['x-custom'], 'kept')
+  t.assert.equal(zlib.gunzipSync(res.rawPayload).toString('utf8'), 'from reply.compress')
+})
+
 test('reply.compress should handle Web ReadableStream', async (t) => {
   t.plan(1)
   const fastify = Fastify()