@fastify/static 5.0.0 → 6.0.0

package/.github/workflows/ci.yml

@@ -1,46 +1,17 @@
 name: CI
-'on':
+
+on:
   push:
     paths-ignore:
-      - docs/**
+      - 'docs/**'
       - '*.md'
   pull_request:
     paths-ignore:
-      - docs/**
+      - 'docs/**'
       - '*.md'
+
 jobs:
   test:
-    runs-on: ${{ matrix.os }}
-    strategy:
-      matrix:
-        node-version:
-          - 10
-          - 12
-          - 14
-          - 16
-        os:
-          - macos-latest
-          - ubuntu-latest
-          - windows-latest
-    steps:
-      - uses: actions/checkout@v3
-      - name: Use Node.js
-        uses: actions/setup-node@v3
-        with:
-          node-version: ${{ matrix.node-version }}
-      - name: Install Dependencies
-        run: |
-          npm install --ignore-scripts
-      - name: Run Tests
-        run: |
-          npm test
-  automerge:
-    needs: test
-    runs-on: ubuntu-latest
-    permissions:
-      pull-requests: write
-      contents: write
-    steps:
-      - uses: fastify/github-action-merge-dependabot@v3
-        with:
-          github-token: ${{ secrets.GITHUB_TOKEN }}
+    uses: fastify/workflows/.github/workflows/plugins-ci.yml@v3
+    with:
+      lint: true

package/README.md

@@ -1,7 +1,7 @@
-# fastify-static
+# @fastify/static
 
 ![CI](https://github.com/fastify/fastify-static/workflows/CI/badge.svg)
-[![NPM version](https://img.shields.io/npm/v/fastify-static.svg?style=flat)](https://www.npmjs.com/package/fastify-static)
+[![NPM version](https://img.shields.io/npm/v/@fastify/static.svg?style=flat)](https://www.npmjs.com/package/@fastify/static)
 [![Known Vulnerabilities](https://snyk.io/test/github/fastify/fastify-static/badge.svg)](https://snyk.io/test/github/fastify/fastify-static)
 [![js-standard-style](https://img.shields.io/badge/code%20style-standard-brightgreen.svg?style=flat)](https://standardjs.com/)
 
@@ -12,7 +12,7 @@ Please refer to [this branch](https://github.com/fastify/fastify-static/tree/1.x
 
 ## Install
 
-`npm install --save fastify-static`
+`npm install --save @fastify/static`
 
 ## Usage
 
@@ -20,7 +20,7 @@ Please refer to [this branch](https://github.com/fastify/fastify-static/tree/1.x
 const fastify = require('fastify')()
 const path = require('path')
 
-fastify.register(require('fastify-static'), {
+fastify.register(require('@fastify/static'), {
   root: path.join(__dirname, 'public'),
   prefix: '/public/', // optional: default '/'
 })
@@ -42,7 +42,7 @@ fastify.get('/another/path', function (req, reply) {
 
 ```js
 const fastify = require('fastify')()
-const fastifyStatic = require('fastify-static')
+const fastifyStatic = require('@fastify/static')
 const path = require('path')
 // first plugin
 fastify.register(fastifyStatic, {
@@ -64,7 +64,7 @@ fastify.register(fastifyStatic, {
 const fastify = require('fastify')()
 const path = require('path')
 
-fastify.register(require('fastify-static'), {
+fastify.register(require('@fastify/static'), {
   root: path.join(__dirname, 'public'),
   prefix: '/public/', // optional: default '/'
 })
@@ -145,7 +145,7 @@ You're able to alter this options when calling `reply.download('filename.html',
 
 Default: `false`
 
-If set to `true`, `fastify-static` redirects to the directory with a trailing slash.
+If set to `true`, `@fastify/static` redirects to the directory with a trailing slash.
 
 This option cannot be set to `true` with `wildcard` set to `false` on a server
 with `ignoreTrailingSlash` set to `true`.
@@ -157,9 +157,9 @@ slash will trigger your app's 404 handler using `reply.callNotFound()`.
 
 Default: `true`
 
-If set to `true`, `fastify-static` adds a wildcard route to serve files.
-If set to `false`, `fastify-static` globs the filesystem for all defined
-files in the served folder (`${root}/**/*`), and just creates the routes needed for
+If set to `true`, `@fastify/static` adds a wildcard route to serve files.
+If set to `false`, `@fastify/static` globs the filesystem for all defined
+files in the served folder (`${root}/**/**`), and just creates the routes needed for
 those and it will not serve the newly added file on the filesystem.
 
 The default options of https://www.npmjs.com/package/glob are applied
@@ -196,7 +196,7 @@ Note: Multi-root is not supported within the `list` option.
 **Example:**
 
 ```js
-fastify.register(require('fastify-static'), {
+fastify.register(require('@fastify/static'), {
   root: path.join(__dirname, 'public'),
   prefix: '/public/',
   index: false
@@ -235,7 +235,7 @@ will return the response as json independent of `list.format`.
 **Example:**
 
 ```js
-fastify.register(require('fastify-static'), {
+fastify.register(require('@fastify/static'), {
   root: path.join(__dirname, 'public'),
   prefix: '/public/',
   list: {
@@ -290,7 +290,7 @@ Note: if a file with the same name exists, the actual file is sent.
 **Example:**
 
 ```js
-fastify.register(require('fastify-static'), {
+fastify.register(require('@fastify/static'), {
   root: path.join(__dirname, '/static'),
   prefix: '/public',
   prefixAvoidTrailingSlash: true,
@@ -408,7 +408,7 @@ If you would just like to use the reply decorator and not serve whole directorie
 #### Disabling reply decorator
 
 The reply object is decorated with a `sendFile` function by default.  If you want to
-disable this, pass the option `{ decorateReply: false }`.  If fastify-static is
+disable this, pass the option `{ decorateReply: false }`.  If @fastify/static is
 registered to multiple prefixes in the same route only one can initialize reply
 decorators.
 

package/example/server-compress.js

@@ -5,7 +5,7 @@ const fastify = require('fastify')({ logger: { level: 'trace' } })
 
 fastify
   // Compress everything.
-  .register(require('fastify-compress'), { threshold: 0 })
+  .register(require('@fastify/compress'), { threshold: 0 })
   .register(require('../'), {
     // An absolute path containing static files to serve.
     root: path.join(__dirname, '/public')

package/index.d.ts

@@ -58,7 +58,7 @@ interface SendOptions {
   etag?: boolean;
   extensions?: string[];
   immutable?: boolean;
-  index?: string[] | false;
+  index?: string[] | string | false;
   lastModified?: boolean;
   maxAge?: string | number;
 }
@@ -88,11 +88,11 @@ export interface FastifyStaticOptions extends SendOptions {
   etag?: boolean;
   extensions?: string[];
   immutable?: boolean;
-  index?: string[] | false;
+  index?: string[] | string | false;
   lastModified?: boolean;
   maxAge?: string | number;
 }
 
-declare const fastifyStatic: FastifyPluginCallback<FastifyStaticOptions>
+export declare const fastifyStatic: FastifyPluginCallback<FastifyStaticOptions>
 
 export default fastifyStatic;

package/index.js

@@ -121,6 +121,7 @@ async function fastifyStatic (fastify, opts) {
 
     wrap.getHeader = reply.getHeader.bind(reply)
     wrap.setHeader = reply.header.bind(reply)
+    wrap.removeHeader = () => {}
     wrap.finished = false
 
     Object.defineProperty(wrap, 'filename', {
@@ -274,7 +275,7 @@ async function fastifyStatic (fastify, opts) {
     schema: {
       hide: typeof opts.schemaHide !== 'undefined' ? opts.schemaHide : true
     },
-    errorHandler: fastify.errorHandler ? errorHandler : undefined
+    errorHandler
   }
 
   if (opts.decorateReply !== false) {
@@ -326,7 +327,7 @@ async function fastifyStatic (fastify, opts) {
         })
       }
     } else {
-      const globPattern = '**/*'
+      const globPattern = '**/**'
       const indexDirs = new Map()
       const routes = new Set()
 
@@ -338,7 +339,7 @@ async function fastifyStatic (fastify, opts) {
           file = file
             .replace(rootPath.replace(/\\/g, '/'), '')
             .replace(/^\//, '')
-          const route = encodeURI(prefix + file).replace(/\/\//g, '/')
+          const route = (prefix + file).replace(/\/\//g, '/')
           if (routes.has(route)) {
             continue
           }
@@ -449,15 +450,21 @@ function getContentType (path) {
 }
 
 function findIndexFile (pathname, root, indexFiles = ['index.html']) {
-  return indexFiles.find(filename => {
-    const p = path.join(root, pathname, filename)
-    try {
-      const stats = statSync(p)
-      return !stats.isDirectory()
-    } catch (e) {
-      return false
-    }
-  })
+  // TODO remove istanbul ignore
+  /* istanbul ignore else */
+  if (Array.isArray(indexFiles)) {
+    return indexFiles.find(filename => {
+      const p = path.join(root, pathname, filename)
+      try {
+        const stats = statSync(p)
+        return !stats.isDirectory()
+      } catch (e) {
+        return false
+      }
+    })
+  }
+  /* istanbul ignore next */
+  return false
 }
 
 // Adapted from https://github.com/fastify/fastify-compress/blob/665e132fa63d3bf05ad37df3c20346660b71a857/index.js#L451
@@ -504,6 +511,6 @@ function getRedirectUrl (url) {
 }
 
 module.exports = fp(fastifyStatic, {
-  fastify: '3.x',
-  name: 'fastify-static'
+  fastify: '4.x',
+  name: '@fastify/static'
 })

package/package.json

@@ -1,17 +1,17 @@
 {
   "name": "@fastify/static",
-  "version": "5.0.0",
+  "version": "6.0.0",
   "description": "Plugin for serving static files as fast as possible.",
   "main": "index.js",
   "types": "index.d.ts",
   "scripts": {
     "lint": "standard | snazzy",
     "lint:fix": "standard --fix",
-    "unit": "tap --no-check-coverage test/*.test.js",
+    "unit": "tap test/*.test.js",
     "typescript": "tsd",
     "test": "npm run lint && npm run unit && npm run typescript",
     "example": "node example/server.js",
-    "coverage": "tap --cov --coverage-report=html --no-check-coverage test",
+    "coverage": "tap --cov --coverage-report=html test",
     "coveralls": "tap test/*test.js  test/*/*.test.js --cov"
   },
   "repository": {
@@ -35,17 +35,17 @@
     "glob": "^7.1.4",
     "p-limit": "^3.1.0",
     "readable-stream": "^3.4.0",
-    "send": "^0.17.1"
+    "send": "^0.18.0"
   },
   "devDependencies": {
+    "@fastify/compress": "^6.0.0",
     "@types/node": "^17.0.0",
     "@typescript-eslint/eslint-plugin": "^2.29.0",
     "@typescript-eslint/parser": "^2.29.0",
     "concat-stream": "^2.0.0",
     "coveralls": "^3.0.4",
     "eslint-plugin-typescript": "^0.14.0",
-    "fastify": "^3.7.0",
-    "fastify-compress": "^4.0.0",
+    "fastify": "^4.0.0-rc.2",
     "handlebars": "^4.7.6",
     "pre-commit": "^1.2.2",
     "proxyquire": "^2.1.0",

package/test/dir-list.test.js

@@ -19,7 +19,7 @@ const helper = {
     const fastify = Fastify()
     fastify.register(mock, options)
     t.teardown(fastify.close.bind(fastify))
-    fastify.listen(0, err => {
+    fastify.listen({ port: 0 }, err => {
       t.error(err)
       fastify.server.unref()
       f('http://localhost:' + fastify.server.address().port)
@@ -102,7 +102,7 @@ t.test('dir list wrong options', t => {
   for (const case_ of cases) {
     const fastify = Fastify()
     fastify.register(fastifyStatic, case_.options)
-    fastify.listen(0, err => {
+    fastify.listen({ port: 0 }, err => {
       t.equal(err.message, case_.error.message)
       fastify.server.unref()
     })

package/test/static-symbolic-link/origin/subdir/subdir/index.html

@@ -0,0 +1,3 @@
+<html>
+  <body>index</body>
+</html>

package/test/static.test.js

@@ -9,8 +9,7 @@ const http = require('http')
 const t = require('tap')
 const simple = require('simple-get')
 const Fastify = require('fastify')
-const { kErrorHandler } = require('fastify/lib/symbols')
-const compress = require('fastify-compress')
+const compress = require('@fastify/compress')
 const concat = require('concat-stream')
 const pino = require('pino')
 const proxyquire = require('proxyquire')
@@ -81,7 +80,7 @@ t.test('register /static prefixAvoidTrailingSlash', t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -241,7 +240,7 @@ t.test('register /static', (t) => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -386,7 +385,7 @@ t.test('register /static/', t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -565,7 +564,7 @@ t.test('register /static and /static2', (t) => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -642,7 +641,7 @@ t.test('payload.filename is set', (t) => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -687,14 +686,14 @@ t.test('error responses can be customized with fastify.setErrorHandler()', t =>
   const fastify = Fastify()
 
   fastify.setErrorHandler(function errorHandler (err, request, reply) {
-    reply.type('text/plain').send(err.status + ' Custom error message')
+    reply.code(403).type('text/plain').send(err.status + ' Custom error message')
   })
 
   fastify.register(fastifyStatic, pluginOptions)
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     fastify.server.unref()
@@ -732,7 +731,7 @@ t.test('not found responses can be customized with fastify.setNotFoundHandler()'
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     fastify.server.unref()
@@ -773,7 +772,7 @@ t.test('fastify.setNotFoundHandler() is called for dotfiles when when send is co
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     fastify.server.unref()
@@ -814,7 +813,7 @@ t.test('serving disabled', (t) => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -871,7 +870,7 @@ t.test('sendFile', (t) => {
     return reply.sendFile('/index.html', { maxAge })
   })
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -954,7 +953,7 @@ t.test('sendFile disabled', (t) => {
     }
   })
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -983,7 +982,7 @@ t.test('allowedPath option', (t) => {
   }
   const fastify = Fastify()
   fastify.register(fastifyStatic, pluginOptions)
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -1072,7 +1071,7 @@ t.test('download', (t) => {
     })
   })
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -1190,7 +1189,7 @@ t.test('sendFile disabled', (t) => {
     }
   })
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -1230,7 +1229,7 @@ t.test('download disabled', (t) => {
     }
   })
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -1272,7 +1271,7 @@ t.test('root not found warning', (t) => {
   )
   const fastify = Fastify({ logger })
   fastify.register(fastifyStatic, pluginOptions)
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
     fastify.server.unref()
     destination.end()
@@ -1329,7 +1328,7 @@ t.test('setHeaders option', (t) => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -1360,7 +1359,7 @@ t.test('maxAge option', (t) => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -1499,7 +1498,7 @@ t.test('register no prefix', (t) => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -1607,7 +1606,7 @@ t.test('with fastify-compress', t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     t.test('deflate', function (t) {
@@ -1643,7 +1642,7 @@ t.test('with fastify-compress', t => {
   })
 })
 t.test('register /static/ with schemaHide true', t => {
-  t.plan(4)
+  t.plan(5)
 
   const pluginOptions = {
     root: path.join(__dirname, '/static'),
@@ -1661,7 +1660,7 @@ t.test('register /static/ with schemaHide true', t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -1683,7 +1682,7 @@ t.test('register /static/ with schemaHide true', t => {
 })
 
 t.test('register /static/ with schemaHide false', t => {
-  t.plan(4)
+  t.plan(5)
 
   const pluginOptions = {
     root: path.join(__dirname, '/static'),
@@ -1701,7 +1700,7 @@ t.test('register /static/ with schemaHide false', t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -1723,7 +1722,7 @@ t.test('register /static/ with schemaHide false', t => {
 })
 
 t.test('register /static/ without schemaHide', t => {
-  t.plan(4)
+  t.plan(5)
 
   const pluginOptions = {
     root: path.join(__dirname, '/static'),
@@ -1740,7 +1739,7 @@ t.test('register /static/ without schemaHide', t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -1768,16 +1767,12 @@ t.test('fastify with exposeHeadRoutes', t => {
     root: path.join(__dirname, '/static'),
     wildcard: false
   }
-  const fastify = Fastify({ exposeHeadRoutes: true })
+  const fastify = Fastify()
   fastify.register(fastifyStatic, pluginOptions)
 
-  fastify.get('/*', (request, reply) => {
-    reply.send({ hello: 'world' })
-  })
-
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     fastify.server.unref()
@@ -1813,7 +1808,7 @@ t.test('register with wildcard false', t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -1957,7 +1952,7 @@ t.test('register with wildcard string on multiple root paths', (t) => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.ok(err)
 
     fastify.server.unref()
@@ -1981,7 +1976,7 @@ t.test('register with wildcard false and alternative index', t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -2134,7 +2129,7 @@ t.test('register /static with wildcard false and alternative index', t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -2292,7 +2287,7 @@ t.test('register /static with redirect true', t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(3001, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     fastify.server.unref()
@@ -2417,7 +2412,7 @@ t.test('register /static with redirect true and wildcard false', t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(3001, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     fastify.server.unref()
@@ -2553,7 +2548,7 @@ t.test('trailing slash behavior with redirect = false', (t) => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
 
     const host = 'http://localhost:' + fastify.server.address().port
@@ -2636,7 +2631,7 @@ t.test('if dotfiles are properly served according to plugin options', (t) => {
     fastify.register(fastifyStatic, pluginOptions)
 
     t.teardown(fastify.close.bind(fastify))
-    fastify.listen(0, (err) => {
+    fastify.listen({ port: 0 }, (err) => {
       t.error(err)
 
       simple.concat({
@@ -2663,7 +2658,7 @@ t.test('if dotfiles are properly served according to plugin options', (t) => {
     fastify.register(fastifyStatic, pluginOptions)
 
     t.teardown(fastify.close.bind(fastify))
-    fastify.listen(0, (err) => {
+    fastify.listen({ port: 0 }, (err) => {
       t.error(err)
 
       simple.concat({
@@ -2689,7 +2684,7 @@ t.test('if dotfiles are properly served according to plugin options', (t) => {
     fastify.register(fastifyStatic, pluginOptions)
 
     t.teardown(fastify.close.bind(fastify))
-    fastify.listen(0, (err) => {
+    fastify.listen({ port: 0 }, (err) => {
       t.error(err)
 
       simple.concat({
@@ -2722,7 +2717,7 @@ t.test('register with failing glob handler', (t) => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     fastify.server.unref()
     t.ok(err)
     t.end()
@@ -2748,7 +2743,7 @@ t.test(
     fastify.register(fastifyStatic, pluginOptions)
 
     t.teardown(fastify.close.bind(fastify))
-    fastify.listen(0, (err) => {
+    fastify.listen({ port: 0 }, (err) => {
       fastify.server.unref()
       t.ok(err)
       t.end()
@@ -2774,7 +2769,7 @@ t.test('inject support', async (t) => {
 })
 
 t.test('routes should use custom errorHandler premature stream close', t => {
-  t.plan(4)
+  t.plan(5)
 
   const pluginOptions = {
     root: path.join(__dirname, '/static'),
@@ -2809,7 +2804,7 @@ t.test('routes should use custom errorHandler premature stream close', t => {
 })
 
 t.test('routes should fallback to default errorHandler', t => {
-  t.plan(4)
+  t.plan(5)
 
   const pluginOptions = {
     root: path.join(__dirname, '/static'),
@@ -2845,44 +2840,6 @@ t.test('routes should fallback to default errorHandler', t => {
   })
 })
 
-t.test('routes use default errorHandler when fastify.errorHandler is not defined', t => {
-  t.plan(4)
-
-  const pluginOptions = {
-    root: path.join(__dirname, '/static'),
-    prefix: '/static/'
-  }
-
-  const fastify = Fastify()
-  fastify[kErrorHandler] = undefined // simulate old fastify version
-
-  fastify.addHook('onRoute', function (routeOptions) {
-    t.notOk(routeOptions.errorHandler instanceof Function)
-
-    routeOptions.preHandler = (request, reply, done) => {
-      const fakeError = new Error()
-      fakeError.code = 'SOMETHING_ELSE'
-      done(fakeError)
-    }
-  })
-
-  fastify.register(fastifyStatic, pluginOptions)
-  t.teardown(fastify.close.bind(fastify))
-
-  fastify.inject({
-    method: 'GET',
-    url: '/static/index.html'
-  }, (err, response) => {
-    t.error(err)
-    t.same(JSON.parse(response.payload), {
-      statusCode: 500,
-      code: 'SOMETHING_ELSE',
-      error: 'Internal Server Error',
-      message: ''
-    })
-  })
-})
-
 t.test('precent encoded URLs in glob mode', (t) => {
   t.plan(4)
 
@@ -2896,7 +2853,7 @@ t.test('precent encoded URLs in glob mode', (t) => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     t.error(err)
     fastify.server.unref()
 
@@ -2927,7 +2884,7 @@ t.test('register /static and /static2 without wildcard', t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     fastify.server.unref()
@@ -3463,7 +3420,7 @@ t.test('should not redirect to protocol-relative locations', (t) => {
     redirect: true
   })
   t.teardown(fastify.close.bind(fastify))
-  fastify.listen(0, (err, address) => {
+  fastify.listen({ port: 0 }, (err, address) => {
     t.error(err)
     urls.forEach(([testUrl, expected, status]) => {
       const req = http.request(url.parse(address + testUrl), res => {
@@ -3480,3 +3437,77 @@ t.test('should not redirect to protocol-relative locations', (t) => {
     })
   })
 })
+
+t.test('should not serve index if option is `false`', (t) => {
+  t.plan(3)
+
+  const pluginOptions = {
+    root: path.join(__dirname, '/static'),
+    prefix: '/static',
+    index: false
+  }
+  const fastify = Fastify()
+  fastify.register(fastifyStatic, pluginOptions)
+
+  t.teardown(fastify.close.bind(fastify))
+
+  fastify.listen({ port: 0 }, (err) => {
+    t.error(err)
+
+    fastify.server.unref()
+
+    t.test('/static/index.html', (t) => {
+      t.plan(3 + GENERIC_RESPONSE_CHECK_COUNT)
+      simple.concat({
+        method: 'GET',
+        url: 'http://localhost:' + fastify.server.address().port + '/static/index.html'
+      }, (err, response, body) => {
+        t.error(err)
+        t.equal(response.statusCode, 200)
+        t.equal(body.toString(), indexContent)
+        genericResponseChecks(t, response)
+      })
+    })
+
+    t.test('/static', (t) => {
+      t.plan(2 + GENERIC_ERROR_RESPONSE_CHECK_COUNT)
+      simple.concat({
+        method: 'GET',
+        url: 'http://localhost:' + fastify.server.address().port + '/static'
+      }, (err, response, body) => {
+        t.error(err)
+        t.equal(response.statusCode, 404)
+        genericErrorResponseChecks(t, response)
+      })
+    })
+  })
+})
+
+t.test('should follow symbolic link without wildcard', (t) => {
+  t.plan(5)
+  const fastify = Fastify()
+  fastify.register(fastifyStatic, {
+    root: path.join(__dirname, '/static-symbolic-link'),
+    wildcard: false
+  })
+  t.teardown(fastify.close.bind(fastify))
+  fastify.listen({ port: 0 }, (err) => {
+    t.error(err)
+
+    simple.concat({
+      method: 'GET',
+      url: 'http://localhost:' + fastify.server.address().port + '/origin/subdir/subdir/index.html'
+    }, (err, response) => {
+      t.error(err)
+      t.equal(response.statusCode, 200)
+    })
+
+    simple.concat({
+      method: 'GET',
+      url: 'http://localhost:' + fastify.server.address().port + '/dir/symlink/subdir/subdir/index.html'
+    }, (err, response) => {
+      t.error(err)
+      t.equal(response.statusCode, 200)
+    })
+  })
+})

package/test/types/index.ts

@@ -28,7 +28,8 @@ const options: FastifyStaticOptions = {
 }
 
 expectError<FastifyStaticOptions>({
-  wlidcard: '**/*'
+  root: '',
+  wildcard: '**/**'
 })
 
 appWithImplicitHttp
@@ -103,3 +104,14 @@ noIndexApp
       reply.send('<h1>fastify-static</h1>')
     })
 })
+
+const defaultIndexApp = fastify()
+options.index = 'index.html'
+
+defaultIndexApp
+  .register(fastifyStatic, options)
+  .after(() => {
+    defaultIndexApp.get('/', (request, reply) => {
+      reply.send('<h1>fastify-static</h1>')
+    })
+  })