@fastify/static 8.0.0 → 8.0.2

package/index.js

@@ -350,7 +350,11 @@ async function fastifyStatic (fastify, opts) {
         break
       }
       case 'file': {
-        reply.code(statusCode)
+        // reply.raw.statusCode by default 200
+        // when ever the user changed it, we respect the status code
+        // otherwise use send provided status code
+        const newStatusCode = reply.statusCode !== 200 ? reply.statusCode : statusCode
+        reply.code(newStatusCode)
         if (setHeaders !== undefined) {
           setHeaders(reply.raw, metadata.path, metadata.stat)
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fastify/static",
-  "version": "8.0.0",
+  "version": "8.0.2",
   "description": "Plugin for serving static files as fast as possible.",
   "main": "index.js",
   "type": "commonjs",
@@ -42,7 +42,7 @@
     "@types/node": "^22.0.0",
     "concat-stream": "^2.0.0",
     "eslint": "^9.9.0",
-    "fastify": "^5.0.0-alpha.4",
+    "fastify": "^5.0.0",
     "neostandard": "^0.11.3",
     "pino": "^9.1.0",
     "proxyquire": "^2.1.3",

package/test/static.test.js

@@ -4023,3 +4023,41 @@ t.test('content-length in head route should not return zero when using wildcard'
     })
   })
 })
+
+t.test('respect the .code when using with sendFile', t => {
+  t.plan(6)
+
+  const pluginOptions = {
+    root: path.join(__dirname, '/static')
+  }
+  const fastify = Fastify()
+
+  fastify.register(fastifyStatic, pluginOptions)
+
+  fastify.get('/custom', (_, reply) => {
+    return reply.code(404).type('text/html').sendFile('foo.html')
+  })
+
+  t.teardown(fastify.close.bind(fastify))
+
+  fastify.listen({ port: 0 }, err => {
+    t.error(err)
+
+    fastify.server.unref()
+
+    const file = fs.readFileSync(path.join(__dirname, '/static/foo.html'))
+    const contentLength = Buffer.byteLength(file).toString()
+
+    simple.concat({
+      method: 'HEAD',
+      url: 'http://localhost:' + fastify.server.address().port + '/custom',
+      followRedirect: false
+    }, (err, response, body) => {
+      t.error(err)
+      t.equal(response.statusCode, 404)
+      t.equal(response.headers['content-type'], 'text/html; charset=UTF-8')
+      t.equal(response.headers['content-length'], contentLength)
+      t.equal(body.toString(), '')
+    })
+  })
+})