fastify 5.1.0 → 5.2.0

package/docs/Reference/Server.md

@@ -24,6 +24,7 @@ describes the properties available in that options object.
   - [`onProtoPoisoning`](#onprotopoisoning)
   - [`onConstructorPoisoning`](#onconstructorpoisoning)
   - [`logger`](#logger)
+  - [`loggerInstance`](#loggerInstance)
   - [`disableRequestLogging`](#disablerequestlogging)
   - [`serverFactory`](#serverfactory)
   - [`caseSensitive`](#casesensitive)
@@ -105,7 +106,7 @@ describes the properties available in that options object.
 
 An object used to configure the server's listening socket. The options
 are the same as the Node.js core [`createServer`
-method](https://nodejs.org/dist/latest-v14.x/docs/api/http.html#http_http_createserver_options_requestlistener).
+method](https://nodejs.org/docs/latest-v20.x/api/http.html#httpcreateserveroptions-requestlistener).
 
 This option is ignored if options [`http2`](#factory-http2) or
 [`https`](#factory-https) are set.
@@ -116,7 +117,7 @@ This option is ignored if options [`http2`](#factory-http2) or
 + Default: `false`
 
 If `true` Node.js core's
-[HTTP/2](https://nodejs.org/dist/latest-v14.x/docs/api/http2.html) module is
+[HTTP/2](https://nodejs.org/dist/latest-v20.x/docs/api/http2.html) module is
 used for binding the socket.
 
 ### `https`
@@ -126,7 +127,7 @@ used for binding the socket.
 
 An object used to configure the server's listening socket for TLS. The options
 are the same as the Node.js core [`createServer`
-method](https://nodejs.org/dist/latest-v14.x/docs/api/https.html#https_https_createserver_options_requestlistener).
+method](https://nodejs.org/dist/latest-v20.x/docs/api/https.html#https_https_createserver_options_requestlistener).
 When this property is `null`, the socket will not be configured for TLS.
 
 This option also applies when the [`http2`](#factory-http2) option is set.
@@ -152,7 +153,7 @@ Defines the server keep-alive timeout in milliseconds. See documentation for
 [`server.keepAliveTimeout`
 property](https://nodejs.org/api/http.html#http_server_keepalivetimeout) to
 understand the effect of this option. This option only applies when HTTP/1 is in
-use. 
+use.
 
 When `serverFactory` option is specified this option is ignored.
 
@@ -203,7 +204,7 @@ ignored.
 Defines the maximum number of milliseconds for receiving the entire request from
 the client. See [`server.requestTimeout`
 property](https://nodejs.org/dist/latest/docs/api/http.html#http_server_requesttimeout)
-to understand the effect of this option. 
+to understand the effect of this option.
 
 When `serverFactory` option is specified, this option is ignored.
 It must be set to a non-zero value (e.g. 120 seconds) to protect against potential
@@ -329,9 +330,6 @@ The possible values this property may have are:
 + Default: `false`. The logger is disabled. All logging methods will point to a
   null logger [abstract-logging](https://npm.im/abstract-logging) instance.
 
-+ `pinoInstance`: a previously instantiated instance of Pino. The internal
-  logger will point to this instance.
-
 + `object`: a standard Pino [options
   object](https://github.com/pinojs/pino/blob/c77d8ec5ce/docs/API.md#constructor).
   This will be passed directly to the Pino constructor. If the following
@@ -351,9 +349,15 @@ The possible values this property may have are:
         ```
       Any user-supplied serializer will override the default serializer of the
       corresponding property.
-+ `loggerInstance`: a custom logger instance. The logger must conform to the
-  Pino interface by having the following methods: `info`, `error`, `debug`,
-  `fatal`, `warn`, `trace`, `child`. For example:
+
+### `loggerInstance`
+<a id="factory-logger-instance"></a>
+
++ Default: `null`
+
+A custom logger instance. The logger must be a Pino instance or conform to the
+Pino interface by having the following methods: `info`, `error`, `debug`,
+`fatal`, `warn`, `trace`, `child`. For example:
   ```js
   const pino = require('pino')();
 
@@ -387,12 +391,12 @@ attaching custom `onRequest` and `onResponse` hooks.
 
 The other log entries that will be disabled are:
 - an error log written by the default `onResponse` hook on reply callback errors
-- the error and info logs written by the `defaultErrorHandler` 
+- the error and info logs written by the `defaultErrorHandler`
 on error management
-- the info log written by the `fourOhFour` handler when a 
+- the info log written by the `fourOhFour` handler when a
 non existent route is requested
 
-Other log messages emitted by Fastify will stay enabled, 
+Other log messages emitted by Fastify will stay enabled,
 like deprecation warnings and messages
 emitted when requests are received while the server is closing.
 
@@ -456,7 +460,7 @@ Please note that setting this option to `false` goes against
 
 By setting `caseSensitive` to `false`, all paths will be matched as lowercase,
 but the route parameters or wildcards will maintain their original letter
-casing. 
+casing.
 This option does not affect query strings, please refer to
 [`querystringParser`](#querystringparser) to change their handling.
 
@@ -493,7 +497,7 @@ Setting `requestIdHeader` to `true` will set the `requestIdHeader` to
 Setting `requestIdHeader` to a non-empty string will use
 the specified string as the `requestIdHeader`.
 By default `requestIdHeader` is set to `false` and will immediately use [genReqId](#genreqid).
-Setting `requestIdHeader` to an empty String (`""`) will set the 
+Setting `requestIdHeader` to an empty String (`""`) will set the
 requestIdHeader to `false`.
 
 + Default: `false`
@@ -827,7 +831,7 @@ is an instance-wide configuration.
 // @param {object} req The raw Node.js HTTP request, not the `FastifyRequest` object.
 // @this Fastify The root Fastify instance (not an encapsulated instance).
 // @returns {string} The path that the request should be mapped to.
-function rewriteUrl (req) { 
+function rewriteUrl (req) {
   if (req.url === '/hi') {
     this.log.debug({ originalUrl: req.url, url: '/hello' }, 'rewrite url');
     return '/hello'
@@ -948,7 +952,7 @@ Starts the server and internally waits for the `.ready()` event. The signature
 is `.listen([options][, callback])`. Both the `options` object and the
 `callback` parameters extend the [Node.js
 core](https://nodejs.org/api/net.html#serverlistenoptions-callback) options
-object. Thus, all core options are available with the following additional 
+object. Thus, all core options are available with the following additional
 Fastify specific options:
 
 ### `listenTextResolver`
@@ -956,13 +960,13 @@ Fastify specific options:
 
 Set an optional resolver for the text to log after server has been successfully
 started.
-It is possible to override the default `Server listening at [address]` log 
+It is possible to override the default `Server listening at [address]` log
 entry using this option.
 
 ```js
-server.listen({ 
-  port: 9080, 
-  listenTextResolver: (address) => { return `Prometheus metrics server is listening at ${address}` } 
+server.listen({
+  port: 9080,
+  listenTextResolver: (address) => { return `Prometheus metrics server is listening at ${address}` }
 })
 ```
 
@@ -1090,7 +1094,7 @@ Method to add routes to the server, it also has shorthand functions, check
 <a id="hasRoute"></a>
 
 Method to check if a route is already registered to the internal router. It
-expects an object as the payload. `url` and `method` are mandatory fields. It 
+expects an object as the payload. `url` and `method` are mandatory fields. It
 is possible to also specify `constraints`. The method returns `true` if the
 route is registered or `false` if not.
 
@@ -1110,8 +1114,8 @@ if (routeExists === false) {
 <a id="findRoute"></a>
 
 Method to retrieve a route already registered to the internal router. It
-expects an object as the payload. `url` and `method` are mandatory fields. It 
-is possible to also specify `constraints`. 
+expects an object as the payload. `url` and `method` are mandatory fields. It
+is possible to also specify `constraints`.
 The method returns a route object or `null` if the route cannot be found.
 
 ```js
@@ -1351,7 +1355,7 @@ Set the schema error formatter for all routes. See
 Set the schema serializer compiler for all routes. See
 [#schema-serializer](./Validation-and-Serialization.md#schema-serializer).
 
-> **Note** 
+> **Note**
 > [`setReplySerializer`](#set-reply-serializer) has priority if set!
 
 #### validatorCompiler
@@ -1966,7 +1970,7 @@ The properties that can currently be exposed are:
 - requestIdHeader
 - requestIdLogLabel
 - http2SessionTimeout
-- useSemicolonDelimiter 
+- useSemicolonDelimiter
 
 ```js
 const { readFileSync } = require('node:fs')

package/docs/Reference/TypeScript.md

@@ -182,7 +182,7 @@ route-level `request` object.
    admin"}`
 
 🎉 Good work, now you can define interfaces for each route and have strictly
-typed request and reply instances. Other parts of the Fastify type system rely 
+typed request and reply instances. Other parts of the Fastify type system rely
 on generic properties. Make sure to reference the detailed type system
 documentation below to learn more about what is available.
 
@@ -873,11 +873,11 @@ a more detailed http server walkthrough.
 
 1. Create the following imports from `@types/node` and `fastify`
    ```typescript
-   import fs from 'fs'
-   import path from 'path'
+   import fs from 'node:fs'
+   import path from 'node:path'
    import fastify from 'fastify'
    ```
-2. Perform the following steps before setting up a Fastify HTTPS server 
+2. Perform the following steps before setting up a Fastify HTTPS server
 to create the `key.pem` and `cert.pem` files:
 ```sh
 openssl genrsa -out key.pem
@@ -935,7 +935,7 @@ specified at server instantiation, the custom type becomes available on all
 further instances of the custom type.
 ```typescript
 import fastify from 'fastify'
-import http from 'http'
+import http from 'node:http'
 
 interface customRequest extends http.IncomingMessage {
   mySpecialProp: string
@@ -1123,8 +1123,8 @@ returns `http.IncomingMessage`, otherwise, it returns
 `http2.Http2ServerRequest`.
 
 ```typescript
-import http from 'http'
-import http2 from 'http2'
+import http from 'node:http'
+import http2 from 'node:http2'
 import { RawRequestDefaultExpression } from 'fastify'
 
 RawRequestDefaultExpression<http.Server> // -> http.IncomingMessage
@@ -1183,8 +1183,8 @@ returns `http.ServerResponse`, otherwise, it returns
 `http2.Http2ServerResponse`.
 
 ```typescript
-import http from 'http'
-import http2 from 'http2'
+import http from 'node:http'
+import http2 from 'node:http2'
 import { RawReplyDefaultExpression } from 'fastify'
 
 RawReplyDefaultExpression<http.Server> // -> http.ServerResponse

package/docs/Reference/Validation-and-Serialization.md

@@ -49,16 +49,16 @@ The shared schemas can be reused through the JSON Schema
 [**`$ref`**](https://tools.ietf.org/html/draft-handrews-json-schema-01#section-8)
 keyword. Here is an overview of _how_ references work:
 
-+ `myField: { $ref: '#foo'}` will search for field with `$id: '#foo'` inside the
++ `myField: { $ref: '#foo' }` will search for field with `$id: '#foo'` inside the
   current schema
-+ `myField: { $ref: '#/definitions/foo'}` will search for field
++ `myField: { $ref: '#/definitions/foo' }` will search for field
   `definitions.foo` inside the current schema
-+ `myField: { $ref: 'http://url.com/sh.json#'}` will search for a shared schema
++ `myField: { $ref: 'http://url.com/sh.json#' }` will search for a shared schema
   added with `$id: 'http://url.com/sh.json'`
-+ `myField: { $ref: 'http://url.com/sh.json#/definitions/foo'}` will search for
++ `myField: { $ref: 'http://url.com/sh.json#/definitions/foo' }` will search for
   a shared schema added with `$id: 'http://url.com/sh.json'` and will use the
   field `definitions.foo`
-+ `myField: { $ref: 'http://url.com/sh.json#foo'}` will search for a shared
++ `myField: { $ref: 'http://url.com/sh.json#foo' }` will search for a shared
   schema added with `$id: 'http://url.com/sh.json'` and it will look inside of
   it for object with `$id: '#foo'`
 

package/examples/typescript-server.ts

@@ -11,7 +11,7 @@
  */
 
 import fastify, { FastifyInstance, RouteShorthandOptions } from '../fastify'
-import { Server, IncomingMessage, ServerResponse } from 'http'
+import { Server, IncomingMessage, ServerResponse } from 'node:http'
 
 // Create an http server. We pass the relevant typings for our http version used.
 // By passing types we get correctly typed access to the underlying http objects in routes.

package/fastify.d.ts

@@ -1,7 +1,7 @@
-import * as http from 'http'
-import * as http2 from 'http2'
-import * as https from 'https'
-import { Socket } from 'net'
+import * as http from 'node:http'
+import * as http2 from 'node:http2'
+import * as https from 'node:https'
+import { Socket } from 'node:net'
 
 import { Options as AjvOptions, ValidatorFactory } from '@fastify/ajv-compiler'
 import { FastifyError } from '@fastify/error'

package/fastify.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const VERSION = '5.1.0'
+const VERSION = '5.2.0'
 
 const Avvio = require('avvio')
 const http = require('node:http')

package/lib/error-handler.js

@@ -112,16 +112,16 @@ function fallbackErrorHandler (error, reply, cb) {
     const serializerFn = getSchemaSerializer(reply[kRouteContext], statusCode, reply[kReplyHeaders]['content-type'])
     payload = (serializerFn === false)
       ? serializeError({
-        error: statusCodes[statusCode + ''],
-        code: error.code,
-        message: error.message,
-        statusCode
-      })
+          error: statusCodes[statusCode + ''],
+          code: error.code,
+          message: error.message,
+          statusCode
+        })
       : serializerFn(Object.create(error, {
-        error: { value: statusCodes[statusCode + ''] },
-        message: { value: error.message },
-        statusCode: { value: statusCode }
-      }))
+          error: { value: statusCodes[statusCode + ''] },
+          message: { value: error.message },
+          statusCode: { value: statusCode }
+        }))
   } catch (err) {
     if (!reply.log[kDisableRequestLogging]) {
       // error is always FST_ERR_SCH_SERIALIZATION_BUILD because this is called from route/compileSchemasForSerialization

package/lib/errors.js

@@ -189,7 +189,7 @@ const codes = {
 
   FST_ERR_HOOK_TIMEOUT: createError(
     'FST_ERR_HOOK_TIMEOUT',
-    "A callback for '%s' hook timed out. You may have forgotten to call 'done' function or to resolve a Promise"
+    "A callback for '%s' hook%s timed out. You may have forgotten to call 'done' function or to resolve a Promise"
   ),
 
   /**

package/lib/hooks.js

@@ -98,9 +98,12 @@ function hookRunnerApplication (hookName, boot, server, cb) {
   next()
 
   function exit (err) {
+    const hookFnName = hooks[i - 1]?.name
+    const hookFnFragment = hookFnName ? ` "${hookFnName}"` : ''
+
     if (err) {
       if (err.code === 'AVV_ERR_READY_TIMEOUT') {
-        err = appendStackTrace(err, new FST_ERR_HOOK_TIMEOUT(hookName))
+        err = appendStackTrace(err, new FST_ERR_HOOK_TIMEOUT(hookName, hookFnFragment))
       } else {
         err = AVVIO_ERRORS_MAP[err.code] != null
           ? appendStackTrace(err, new AVVIO_ERRORS_MAP[err.code](err.message))

package/lib/request.js

@@ -117,10 +117,11 @@ function buildRequestWithTrustProxy (R, trustProxy) {
         if (this.ip !== undefined && this.headers['x-forwarded-host']) {
           return getLastEntryInMultiHeaderValue(this.headers['x-forwarded-host'])
         }
-        let host = this.headers.host ?? this.headers[':authority']
-        // support http.requireHostHeader === false
-        if (this.server.server.requireHostHeader === false) host ??= ''
-        return host
+        // the last fallback is used to support the following cases:
+        // 1. support http.requireHostHeader === false
+        // 2. support HTTP/1.0 without Host Header
+        // 3. support headers schema which may remove the Host Header
+        return this.headers.host ?? this.headers[':authority'] ?? ''
       }
     },
     protocol: {
@@ -212,10 +213,11 @@ Object.defineProperties(Request.prototype, {
   },
   host: {
     get () {
-      let host = this.raw.headers.host ?? this.raw.headers[':authority']
-      // support http.requireHostHeader === false
-      if (this.server.server.requireHostHeader === false) host ??= ''
-      return host
+      // the last fallback is used to support the following cases:
+      // 1. support http.requireHostHeader === false
+      // 2. support HTTP/1.0 without Host Header
+      // 3. support headers schema which may remove the Host Header
+      return this.raw.headers.host ?? this.raw.headers[':authority'] ?? ''
     }
   },
   hostname: {
@@ -231,8 +233,7 @@ Object.defineProperties(Request.prototype, {
         return portFromHost
       }
       // now fall back to port from host/:authority header
-      let host = (this.headers.host ?? this.headers[':authority'])
-      if (this.server.server.requireHostHeader === false) host ??= ''
+      const host = (this.headers.host ?? this.headers[':authority'] ?? '')
       const portFromHeader = parseInt(host.split(':').slice(-1)[0])
       if (!isNaN(portFromHeader)) {
         return portFromHeader

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fastify",
-  "version": "5.1.0",
+  "version": "5.2.0",
   "description": "Fast and low overhead web framework, for Node.js",
   "main": "fastify.js",
   "type": "commonjs",
@@ -11,7 +11,6 @@
     "benchmark:parser": "concurrently -k -s first \"node ./examples/benchmark/parser.js\" \"autocannon -c 100 -d 30 -p 10 -i ./examples/benchmark/body.json -H \"content-type:application/jsoff\" -m POST localhost:3000/\"",
     "build:validation": "node build/build-error-serializer.js && node build/build-validation.js",
     "coverage": "c8 --reporter html borp --reporter=./test/test-reporter.mjs --coverage --check-coverage --lines 100 ",
-    "coverage:ci": "c8 --reporter lcov --reporter html borp --reporter=./test/test-reporter.mjs",
     "coverage:ci-check-coverage": "borp --reporter=./test/test-reporter.mjs --coverage --check-coverage --lines 100",
     "lint": "npm run lint:eslint",
     "lint:fix": "eslint --fix",
@@ -168,7 +167,7 @@
     "cross-env": "^7.0.3",
     "eslint": "^9.0.0",
     "fast-json-body": "^1.1.0",
-    "fastify-plugin": "^4.5.1",
+    "fastify-plugin": "^5.0.0",
     "fluent-json-schema": "^5.0.0",
     "h2url": "^0.2.0",
     "http-errors": "^2.0.0",
@@ -201,7 +200,7 @@
     "process-warning": "^4.0.0",
     "proxy-addr": "^2.0.7",
     "rfdc": "^1.3.1",
-    "secure-json-parse": "^2.7.0",
+    "secure-json-parse": "^3.0.1",
     "semver": "^7.6.0",
     "toad-cache": "^3.7.0"
   },

package/test/{allowUnsafeRegex.test.js → allow-unsafe-regex.test.js}

@@ -24,7 +24,7 @@ test('allow unsafe regex', (t, done) => {
       url: 'http://localhost:' + fastify.server.address().port + '/1234'
     }, (err, response, body) => {
       t.assert.ifError(err)
-      t.assert.equal(response.statusCode, 200)
+      t.assert.strictEqual(response.statusCode, 200)
       t.assert.deepStrictEqual(JSON.parse(body), {
         foo: '1234'
       })
@@ -53,7 +53,7 @@ test('allow unsafe regex not match', (t, done) => {
       url: 'http://localhost:' + fastify.server.address().port + '/a1234'
     }, (err, response, body) => {
       t.assert.ifError(err)
-      t.assert.equal(response.statusCode, 404)
+      t.assert.strictEqual(response.statusCode, 404)
       done()
     })
   })
@@ -111,8 +111,8 @@ test('allow unsafe regex allow unsafe', (t, done) => {
       url: 'http://localhost:' + fastify.server.address().port + '/1234'
     }, (err, response, body) => {
       t.assert.ifError(err)
-      t.assert.equal(response.statusCode, 200)
-      t.assert.deepEqual(JSON.parse(body), {
+      t.assert.strictEqual(response.statusCode, 200)
+      t.assert.deepStrictEqual(JSON.parse(body), {
         foo: '1234'
       })
       done()

package/test/async-dispose.test.js

@@ -16,6 +16,5 @@ test('async dispose should close fastify', { skip: !('asyncDispose' in Symbol) }
   // the same as syntax sugar for
   // await using app = fastify()
   await fastify[Symbol.asyncDispose]()
-
   t.assert.strictEqual(fastify.server.listening, false)
 })

package/test/async_hooks.test.js

@@ -41,7 +41,7 @@ test('test async hooks', (t, done) => {
       json: true
     }, (err, response, body) => {
       t.assert.ifError(err)
-      t.assert.equal(response.statusCode, 200)
+      t.assert.strictEqual(response.statusCode, 200)
 
       sget({
         method: 'POST',
@@ -52,7 +52,7 @@ test('test async hooks', (t, done) => {
         json: true
       }, (err, response, body) => {
         t.assert.ifError(err)
-        t.assert.equal(response.statusCode, 200)
+        t.assert.strictEqual(response.statusCode, 200)
 
         sget({
           method: 'GET',
@@ -60,9 +60,9 @@ test('test async hooks', (t, done) => {
           json: true
         }, (err, response, body) => {
           t.assert.ifError(err)
-          t.assert.equal(response.statusCode, 200)
+          t.assert.strictEqual(response.statusCode, 200)
           app.close()
-          t.assert.equal(remainingIds.size, 0)
+          t.assert.strictEqual(remainingIds.size, 0)
           done()
         })
       })

package/test/buffer.test.js

@@ -24,8 +24,8 @@ test('Buffer test', async t => {
     })
 
     t.assert.ifError(response.error)
-    t.assert.equal(response.statusCode, 200)
-    t.assert.deepEqual(response.payload.toString(), '{"hello":"world"}')
+    t.assert.strictEqual(response.statusCode, 200)
+    t.assert.deepStrictEqual(response.payload.toString(), '{"hello":"world"}')
   })
 
   await test('should return 400 if the body is empty', async t => {
@@ -41,8 +41,8 @@ test('Buffer test', async t => {
     })
 
     t.assert.ifError(response.error)
-    t.assert.equal(response.statusCode, 400)
-    t.assert.deepEqual(JSON.parse(response.payload.toString()), {
+    t.assert.strictEqual(response.statusCode, 400)
+    t.assert.deepStrictEqual(JSON.parse(response.payload.toString()), {
       error: 'Bad Request',
       code: 'FST_ERR_CTP_EMPTY_JSON_BODY',
       message: 'Body cannot be empty when content-type is set to \'application/json\'',

package/test/build-certificate.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const os = require('os')
+const os = require('node:os')
 const forge = require('node-forge')
 
 // from self-cert module

package/test/bundler/README.md

@@ -1,12 +1,12 @@
 # Bundlers test stack
 
-In some cases, developers bundle their apps for several targets such as serverless applications. 
-Even if it's not recommended by Fastify team; we need to ensure we do not break the build process. 
+In some cases, developers bundle their apps for several targets such as serverless applications.
+Even if it's not recommended by Fastify team; we need to ensure we do not break the build process.
 Please note this might result in features behaving differently, like the version handling check for plugins.
 
 ## Test bundlers
 
-The bundler test stack has been defined separately from the rest of the Unit testing stack because it's not a 
+The bundler test stack has been defined separately from the rest of the Unit testing stack because it's not a
 part of the fastify lib itself. Note that the tests run in CI only on NodeJs LTS version.
 Developers do not need to install every bundler to run unit tests.
 
@@ -23,7 +23,7 @@ stack dependencies. See:
 
 ## Bundler test development
 
-To not break the fastify unit testing stack please name test files like this `*-test.js` and not `*.test.js`, 
+To not break the fastify unit testing stack please name test files like this `*-test.js` and not `*.test.js`,
 otherwise it will be targeted by the regular expression used for unit tests for fastify.
-Tests need to ensure the build process works and the fastify application can be run, 
+Tests need to ensure the build process works and the fastify application can be run,
 no need to go in deep testing unless an issue is raised.

package/test/case-insensitive.test.js

@@ -24,8 +24,8 @@ test('case insensitive', (t, done) => {
       url: 'http://localhost:' + fastify.server.address().port + '/FOO'
     }, (err, response, body) => {
       t.assert.ifError(err)
-      t.assert.equal(response.statusCode, 200)
-      t.assert.deepEqual(JSON.parse(body), {
+      t.assert.strictEqual(response.statusCode, 200)
+      t.assert.deepStrictEqual(JSON.parse(body), {
         hello: 'world'
       })
       done()
@@ -53,8 +53,8 @@ test('case insensitive inject', (t, done) => {
       url: 'http://localhost:' + fastify.server.address().port + '/FOO'
     }, (err, response) => {
       t.assert.ifError(err)
-      t.assert.equal(response.statusCode, 200)
-      t.assert.deepEqual(JSON.parse(response.payload), {
+      t.assert.strictEqual(response.statusCode, 200)
+      t.assert.deepStrictEqual(JSON.parse(response.payload), {
         hello: 'world'
       })
       done()
@@ -71,7 +71,7 @@ test('case insensitive (parametric)', (t, done) => {
   t.after(() => fastify.close())
 
   fastify.get('/foo/:param', (req, reply) => {
-    t.assert.equal(req.params.param, 'bAr')
+    t.assert.strictEqual(req.params.param, 'bAr')
     reply.send({ hello: 'world' })
   })
 
@@ -83,8 +83,8 @@ test('case insensitive (parametric)', (t, done) => {
       url: 'http://localhost:' + fastify.server.address().port + '/FoO/bAr'
     }, (err, response, body) => {
       t.assert.ifError(err)
-      t.assert.equal(response.statusCode, 200)
-      t.assert.deepEqual(JSON.parse(body), {
+      t.assert.strictEqual(response.statusCode, 200)
+      t.assert.deepStrictEqual(JSON.parse(body), {
         hello: 'world'
       })
       done()
@@ -101,7 +101,7 @@ test('case insensitive (wildcard)', (t, done) => {
   t.after(() => fastify.close())
 
   fastify.get('/foo/*', (req, reply) => {
-    t.assert.equal(req.params['*'], 'bAr/baZ')
+    t.assert.strictEqual(req.params['*'], 'bAr/baZ')
     reply.send({ hello: 'world' })
   })
 
@@ -113,8 +113,8 @@ test('case insensitive (wildcard)', (t, done) => {
       url: 'http://localhost:' + fastify.server.address().port + '/FoO/bAr/baZ'
     }, (err, response, body) => {
       t.assert.ifError(err)
-      t.assert.equal(response.statusCode, 200)
-      t.assert.deepEqual(JSON.parse(body), {
+      t.assert.strictEqual(response.statusCode, 200)
+      t.assert.deepStrictEqual(JSON.parse(body), {
         hello: 'world'
       })
       done()

package/test/check.test.js

@@ -72,7 +72,6 @@ const options = {
 }
 
 const handler = (request, reply) => {
-  console.log('in handler')
   if (request.body.id === '400') {
     return reply.status(400).send({
       statusCode: 400,
@@ -124,8 +123,8 @@ test('serialize the response for a Bad Request error, as defined on the schema',
       json: true
     }, (err, response, body) => {
       t.assert.ifError(err)
-      t.assert.equal(response.statusCode, 400)
-      t.assert.deepEqual(body, {
+      t.assert.strictEqual(response.statusCode, 400)
+      t.assert.deepStrictEqual(body, {
         statusCode: 400,
         error: 'Bad Request',
         message: 'body must be object'

package/test/{childLoggerFactory.test.js → child-logger-factory.test.js}

@@ -74,7 +74,7 @@ test('req.log should be the instance returned by the factory', (t, done) => {
   })
 
   fastify.get('/', (req, reply) => {
-    t.assert.equal(req.log, fastify.log)
+    t.assert.strictEqual(req.log, fastify.log)
     req.log.info('log message')
     reply.send()
   })

package/test/client-timeout.test.js

@@ -28,7 +28,7 @@ test('requestTimeout should return 408', (t, done) => {
 
     socket.on('data', c => (data = Buffer.concat([data, c])))
     socket.on('end', () => {
-      t.assert.equal(
+      t.assert.strictEqual(
         data.toString('utf-8'),
         'HTTP/1.1 408 Request Timeout\r\nContent-Length: 71\r\nContent-Type: application/json\r\n\r\n{"error":"Request Timeout","message":"Client Timeout","statusCode":408}'
       )