fastify 3.15.1 → 3.17.0

package/.taprc

@@ -1,9 +1,5 @@
 ts: false
 jsx: false
 flow: false
-check-coverage: false
+check-coverage: true
 coverage: true
-statements: 99
-lines: 99
-branches: 98
-functions: 99

package/docs/Ecosystem.md

@@ -130,6 +130,7 @@ Plugins maintained by the Fastify team are listed under [Core](#core) while plug
 - [`fastify-mongo-memory`](https://github.com/chapuletta/fastify-mongo-memory) Fastify MongoDB in Memory Plugin for testing support.
 - [`fastify-mongoose-api`](https://github.com/jeka-kiselyov/fastify-mongoose-api) Fastify plugin to create REST API methods based on Mongoose MongoDB models.
 - [`fastify-mongoose-driver`](https://github.com/alex-ppg/fastify-mongoose) Fastify Mongoose plugin that connects to a MongoDB via the Mongoose plugin with support for Models.
+- [`fastify-msgpack`](https://github.com/kenriortega/fastify-msgpack) Fastify and MessagePack, together at last. Uses @msgpack/msgpack by default. 
 - [`fastify-multer`](https://github.com/fox1t/fastify-multer) Multer is a plugin for handling multipart/form-data, which is primarily used for uploading files.
 - [`fastify-nats`](https://github.com/mahmed8003/fastify-nats) Plugin to share [NATS](https://nats.io) client across Fastify.
 - [`fastify-no-additional-properties`](https://github.com/greguz/fastify-no-additional-properties) Add `additionalProperties: false` by default to your JSON Schemas.
@@ -151,6 +152,7 @@ Plugins maintained by the Fastify team are listed under [Core](#core) while plug
 - [`fastify-postgraphile`](https://github.com/alemagio/fastify-postgraphile) Plugin to integrate [PostGraphile](https://www.graphile.org/postgraphile/) in a Fastify project.
 - [`fastify-prettier`](https://github.com/hsynlms/fastify-prettier) A Fastify plugin that uses [prettier](https://github.com/prettier/prettier) under the hood to beautify outgoing responses and/or other things in the Fastify server.
 - [`fastify-print-routes`](https://github.com/ShogunPanda/fastify-print-routes) A Fastify plugin that prints all available routes.
+- [`fastify-protobufjs`](https://github.com/kenriortega/fastify-protobufjs) Fastify and protobufjs, together at last. Uses protobufjs by default. 
 - [`fastify-qrcode`](https://github.com/chonla/fastify-qrcode) This plugin utilizes [qrcode](https://github.com/soldair/node-qrcode) to generate QR Code.
 - [`fastify-qs`](https://github.com/webdevium/fastify-qs) A plugin for Fastify that adds support for parsing URL query parameters with [qs](https://github.com/ljharb/qs).
 - [`fastify-raw-body`](https://github.com/Eomm/fastify-raw-body) Add the `request.rawBody` field.

package/docs/LTS.md

@@ -25,6 +25,19 @@ out in this document:
 
 A "month" is defined as 30 consecutive days.
 
+> ## Security Releases and Semver
+>
+> As a consequence of providing long-term support for major releases, there
+> are occasions where we need to release breaking changes as a _minor_
+> version release. Such changes will _always_ be noted in the
+> [release notes](https://github.com/fastify/fastify/releases).
+>
+> To avoid automatically receiving breaking security updates it is possible to use
+> the tilde (`~`) range qualifier. For example, to get patches for the 3.15
+> release, and avoid automatically updating to the 3.16 release, specify
+> the dependency as `"fastify": "~3.15.x"`. This will leave your application vulnerable,
+> so please use with caution.
+
 [semver]: https://semver.org/
 
 <a name="lts-schedule"></a>
@@ -41,12 +54,15 @@ A "month" is defined as 30 consecutive days.
 
 ### CI tested operating systems
 
-Fastify uses GitHub Actions for CI testing, please refer to [GitHub's documentation regarding workflow runners](https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners#supported-runners-and-hardware-resources) for further details on what the latest virtual environment is in relation to the YAML workflow labels below:
+Fastify uses GitHub Actions for CI testing, please refer to
+[GitHub's documentation regarding workflow runners](https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners#supported-runners-and-hardware-resources)
+for further details on what the latest virtual environment is in relation to
+the YAML workflow labels below:
 
 | OS      | YAML Workflow Label    | Package Manager           | Node.js      |
 |---------|------------------------|---------------------------|--------------|
 | Linux   | `ubuntu-latest`        | npm                       | 10,12,14,16  |
-| Linux   | `ubuntu-16.04`         | yarn,pnpm                 | 10,12        |
+| Linux   | `ubuntu-18.04`         | yarn,pnpm                 | 10,12        |
 | Windows | `windows-latest`       | npm                       | 10,12,14,16  |
 | MacOS   | `macos-latest`         | npm                       | 10,12,14,16  |
 

package/docs/Reply.md

@@ -5,6 +5,7 @@
   - [Introduction](#introduction)
   - [.code(statusCode)](#codestatuscode)
   - [.statusCode](#statusCode)
+  - [.server](#server)
   - [.header(key, value)](#headerkey-value)
   - [.headers(object)](#headersobject)
   - [.getHeader(key)](#getheaderkey)
@@ -38,6 +39,7 @@ and properties:
 - `.code(statusCode)` - Sets the status code.
 - `.status(statusCode)` - An alias for `.code(statusCode)`.
 - `.statusCode` - Read and set the HTTP status code.
+- `.server` - A reference to the fastify instance object.
 - `.header(name, value)` - Sets a response header.
 - `.headers(object)` - Sets all the keys of the object as response headers.
 - `.getHeader(name)` - Retrieve value of already set header.
@@ -87,6 +89,20 @@ if (reply.statusCode >= 299) {
 }
 ```
 
+<a name="server"></a>
+### .server
+The Fastify server instance, scoped to the current [encapsulation context](Encapsulation.md).
+
+```js
+fastify.decorate('util', function util () {
+  return 'foo'
+})
+
+fastify.get('/', async function (req, rep) {
+  return rep.server.util() // foo
+})
+```
+
 <a name="header"></a>
 ### .header(key, value)
 Sets a response header. If the value is omitted or undefined, it is coerced

package/docs/Request.md

@@ -9,6 +9,7 @@ Request is a core Fastify object containing the following fields:
 - `headers` - the headers
 - `raw` - the incoming HTTP request from Node core
 - `req` *(deprecated, use `.raw` instead)* - the incoming HTTP request from Node core
+- `server` - The Fastify server instance, scoped to the current [encapsulation context](Encapsulation.md)
 - `id` - the request id
 - `log` - the logger instance of the incoming request
 - `ip` - the IP address of the incoming request
@@ -31,6 +32,7 @@ fastify.post('/:params', options, function (request, reply) {
   console.log(request.params)
   console.log(request.headers)
   console.log(request.raw)
+  console.log(request.server)
   console.log(request.id)
   console.log(request.ip)
   console.log(request.ips)

package/docs/Server.md

@@ -404,6 +404,19 @@ const fastify = require('fastify')({
 })
 ```
 
+<a name="serializer-opts"></a>
+### `serializerOpts`
+
+Customize the options of the default [`fast-json-stringify`](https://github.com/fastify/fast-json-stringify#options) instance that serialize the response's payload:
+
+```js
+const fastify = require('fastify')({
+  serializerOpts: {
+    rounding: 'ceil'
+  }
+})
+```
+
 <a name="http2-session-timeout"></a>
 ### `http2SessionTimeout`
 
@@ -841,7 +854,10 @@ This property can be used to set a function to format errors that happen while t
 
 <a name="schema-controller"></a>
 #### schemaController
-This property can be used to fully manage where the schemas of your application will be stored.
+This property can be used to fully manage: 
+- `bucket`: where the schemas of your application will be stored
+- `compilersFactory`: what module must compile the JSON schemas
+
 It can be useful when your schemas are stored in another data structure that is unknown to Fastify.
 See [issue #2446](https://github.com/fastify/fastify/issues/2446) for an example of what
 this property helps to resolve.
@@ -875,6 +891,45 @@ const fastify = Fastify({
           return allTheSchemaStored
         }
       }
+    },
+
+    /**
+     * The compilers factory let you to fully control the validator and serializer
+     * in the Fastify's lifecycle, providing the encapsulation to your compilers.
+     */
+    compilersFactory: {
+      /**
+       * This factory is called whenever a new validator instance is needed.
+       * It may be called whenever `fastify.register()` is called only if new schemas has been added to the
+       * encapsulation context.
+       * It may receive as input the schemas of the parent context if some schemas has been added.
+       * @param {object} externalSchemas these schemas will be returned by the `bucket.getSchemas()`. Needed to resolve the external references $ref.
+       * @param {object} ajvServerOption the server `ajv` options to build your compilers accordingly 
+       */
+      buildValidator: function factory (externalSchemas, ajvServerOption) {
+        // This factory function must return a schema validator compiler.
+        // See [#schema-validator](Validation-and-Serialization.md#schema-validator) for details.
+        const yourAjvInstance = new Ajv(ajvServerOption.customOptions)
+        return function validatorCompiler ({ schema, method, url, httpPart }) {
+          return yourAjvInstance.compile(schema)
+        }
+      },
+
+      /**
+       * This factory is called whenever a new serializer instance is needed.
+       * It may be called whenever `fastify.register()` is called only if new schemas has been added to the
+       * encapsulation context.
+       * It may receive as input the schemas of the parent context if some schemas has been added.
+       * @param {object} externalSchemas these schemas will be returned by the `bucket.getSchemas()`. Needed to resolve the external references $ref.
+       * @param {object} serializerOptsServerOption the server `serializerOpts` options to build your compilers accordingly 
+       */
+      buildSerializer: function factory (externalSchemas, serializerOptsServerOption) {
+        // This factory function must return a schema serializer compiler.
+        // See [#schema-serializer](Validation-and-Serialization.md#schema-serializer) for details.
+        return function serializerCompiler ({ schema, method, url, httpStatus }) {
+          return data => JSON.stringify(data)
+        }
+      }
     }
   }
 });

package/docs/TypeScript.md

@@ -4,7 +4,7 @@
 
 The Fastify framework is written in vanilla JavaScript, and as such type definitions are not as easy to maintain; however, since version 2 and beyond, maintainers and contributors have put in a great effort to improve the types.
 
-The type system was changed in Fastify version 3. The new type system introduces generic constraining and defaulting, plus a new way to define schema types such as a request body, querystring, and more! As the team works on improving framework and type definition synergy, sometimes parts of the API will not be typed or may be typed incorrectly. We encourage you to **contribute** to help us fill in the gaps. Just make sure to read our [`CONTRIBUTING.md`](../CONTRIBUTING.md) file before getting started to make sure things go smoothly!
+The type system was changed in Fastify version 3. The new type system introduces generic constraining and defaulting, plus a new way to define schema types such as a request body, querystring, and more! As the team works on improving framework and type definition synergy, sometimes parts of the API will not be typed or may be typed incorrectly. We encourage you to **contribute** to help us fill in the gaps. Just make sure to read our [`CONTRIBUTING.md`](https://github.com/fastify/fastify/blob/HEAD/CONTRIBUTING.md) file before getting started to make sure things go smoothly!
 
 > The documentation in this section covers Fastify version 3.x typings
 

package/docs/Validation-and-Serialization.md

@@ -356,7 +356,7 @@ fastify.setValidatorCompiler(({ schema, method, url, httpPart }) => {
   return ajv.compile(schema)
 })
 ```
-_**Note:** If you use a custom instance of any validator (even Ajv), you have to add schemas to the validator instead of Fastify, since Fastify's default validator is no longer used, and astify's `addSchema` method has no idea what validator you are using._
+_**Note:** If you use a custom instance of any validator (even Ajv), you have to add schemas to the validator instead of Fastify, since Fastify's default validator is no longer used, and Fastify's `addSchema` method has no idea what validator you are using._
 
 <a name="using-other-validation-libraries"></a>
 ##### Using other validation libraries

package/examples/asyncawait.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const fastify = require('../fastify')()
+const fastify = require('../fastify')({ logger: true })
 
 const schema = {
   schema: {
@@ -33,5 +33,4 @@ fastify
 
 fastify.listen(3000, err => {
   if (err) throw err
-  console.log(`server listening on ${fastify.server.address().port}`)
 })

package/examples/{hooks-benchmark-async-await.js → benchmark/hooks-benchmark-async-await.js}

@@ -1,6 +1,6 @@
 'use strict'
 
-const fastify = require('../fastify')()
+const fastify = require('../../fastify')({ logger: false })
 
 const opts = {
   schema: {
@@ -41,5 +41,4 @@ fastify.listen(3000, function (err) {
   if (err) {
     throw err
   }
-  console.log(`server listening on ${fastify.server.address().port}`)
 })

package/examples/{hooks-benchmark.js → benchmark/hooks-benchmark.js}

@@ -1,6 +1,6 @@
 'use strict'
 
-const fastify = require('../fastify')()
+const fastify = require('../../fastify')({ logger: false })
 
 const opts = {
   schema: {
@@ -49,5 +49,4 @@ fastify.listen(3000, function (err) {
   if (err) {
     throw err
   }
-  console.log(`server listening on ${fastify.server.address().port}`)
 })

package/examples/benchmark/simple.js

@@ -0,0 +1,30 @@
+'use strict'
+
+const fastify = require('../../fastify')({
+  logger: false
+})
+
+const schema = {
+  schema: {
+    response: {
+      200: {
+        type: 'object',
+        properties: {
+          hello: {
+            type: 'string'
+          }
+        }
+      }
+    }
+  }
+}
+
+fastify
+  .get('/', schema, function (req, reply) {
+    reply
+      .send({ hello: 'world' })
+  })
+
+fastify.listen(3000, (err, address) => {
+  if (err) throw err
+})

package/examples/hooks.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const fastify = require('../fastify')()
+const fastify = require('../fastify')({ logger: true })
 
 const opts = {
   schema: {
@@ -85,5 +85,4 @@ fastify.listen(3000, function (err) {
   if (err) {
     throw err
   }
-  console.log(`server listening on ${fastify.server.address().port}`)
 })

package/examples/http2.js

@@ -7,7 +7,8 @@ const fastify = require('../fastify')({
   https: {
     key: fs.readFileSync(path.join(__dirname, '../test/https/fastify.key')),
     cert: fs.readFileSync(path.join(__dirname, '../test/https/fastify.cert'))
-  }
+  },
+  logger: true
 })
 
 const opts = {
@@ -33,5 +34,4 @@ fastify
 
 fastify.listen(3000, err => {
   if (err) throw err
-  console.log(`server listening on ${fastify.server.address().port}`)
 })

package/examples/https.js

@@ -6,7 +6,8 @@ const fastify = require('../fastify')({
   https: {
     key: fs.readFileSync(path.join(__dirname, '../test/https/fastify.key')),
     cert: fs.readFileSync(path.join(__dirname, '../test/https/fastify.cert'))
-  }
+  },
+  logger: true
 })
 
 const opts = {
@@ -32,5 +33,4 @@ fastify
 
 fastify.listen(3000, err => {
   if (err) throw err
-  console.log(`server listening on ${fastify.server.address().port}`)
 })

package/examples/parser.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const fastify = require('../fastify')()
+const fastify = require('../fastify')({ logger: true })
 const jsonParser = require('fast-json-body')
 const qs = require('qs')
 
@@ -38,5 +38,4 @@ fastify
 
 fastify.listen(3000, err => {
   if (err) throw err
-  console.log(`server listening on ${fastify.server.address().port}`)
 })

package/examples/route-prefix.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const fastify = require('../fastify')()
+const fastify = require('../fastify')({ logger: true })
 
 const opts = {
   schema: {
@@ -35,5 +35,4 @@ fastify.listen(8000, function (err) {
   if (err) {
     throw err
   }
-  console.log(`server listening on ${fastify.server.address().port}`)
 })

package/examples/shared-schema.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const fastify = require('../fastify')()
+const fastify = require('../fastify')({ logger: true })
 
 fastify.addSchema({
   $id: 'https://foo/common.json',
@@ -33,5 +33,4 @@ fastify
 
 fastify.listen(3000, err => {
   if (err) throw err
-  console.log(`server listening on ${fastify.server.address().port}`)
 })

package/examples/simple.js

@@ -27,5 +27,4 @@ fastify
 
 fastify.listen(3000, (err, address) => {
   if (err) throw err
-  fastify.log.info(`server listening on ${address}`)
 })

package/examples/simple.mjs

@@ -2,7 +2,7 @@
 import { fastify } from '../fastify.js'
 
 const app = fastify({
-  logger: false
+  logger: true
 })
 
 const schema = {

package/examples/typescript-server.ts

@@ -20,7 +20,7 @@ const server: FastifyInstance<
   Server,
   IncomingMessage,
   ServerResponse
-> = fastify();
+> = fastify({ logger: true });
 
 // Define interfaces for our request. We can create these automatically
 // off our JSON Schema files (See TypeScript.md) but for the purpose of this
@@ -75,5 +75,4 @@ server.listen(8080, (err, address) => {
     console.error(err);
     process.exit(0);
   }
-  console.log(`Server listening at ${address}`);
 });

package/examples/use-plugin.js

@@ -1,4 +1,4 @@
-const fastify = require('../fastify')()
+const fastify = require('../fastify')({ logger: true })
 
 const opts = {
   schema: {
@@ -22,5 +22,4 @@ fastify.listen(3000, function (err) {
   if (err) {
     throw err
   }
-  console.log(`server listening on ${fastify.server.address().port}`)
 })

package/fastify.d.ts

@@ -9,11 +9,12 @@ import { RawServerBase, RawServerDefault, RawRequestDefaultExpression, RawReplyD
 import { FastifyLoggerInstance, FastifyLoggerOptions } from './types/logger'
 import { FastifyInstance } from './types/instance'
 import { FastifyServerFactory } from './types/serverFactory'
-import * as ajv from 'ajv'
+import { Options as AjvOptions } from '@fastify/ajv-compiler'
 import { FastifyError } from 'fastify-error'
 import { FastifyReply } from './types/reply'
 import { FastifySchemaValidationError } from './types/schema'
 import { ConstructorAction, ProtoAction } from "./types/content-type-parser";
+import { Socket } from 'net'
 
 /**
  * Fastify factory function for the standard fastify http, https, or http2 server instance.
@@ -74,6 +75,15 @@ export type FastifyHttpsOptions<
 
 type FindMyWayVersion<RawServer extends RawServerBase> = RawServer extends http.Server ? HTTPVersion.V1 : HTTPVersion.V2
 
+export interface ConnectionError extends Error {
+  code: string,
+  bytesParsed: number,
+  rawPacket: {
+    type: string,
+    data: number[]
+  }
+}
+
 /**
  * Options for a fastify server instance. Utilizes conditional logic on the generic server parameter to enforce certain https and http2
  */
@@ -116,7 +126,7 @@ export type FastifyServerOptions<
   },
   return503OnClosing?: boolean,
   ajv?: {
-    customOptions?: ajv.Options,
+    customOptions?: AjvOptions,
     plugins?: Function[]
   },
   frameworkErrors?: <RequestGeneric extends RequestGenericInterface = RequestGenericInterface>(
@@ -125,7 +135,11 @@ export type FastifyServerOptions<
     res: FastifyReply<RawServer, RawRequestDefaultExpression<RawServer>, RawReplyDefaultExpression<RawServer>>
   ) => void,
   rewriteUrl?: (req: RawRequestDefaultExpression<RawServer>) => string,
-  schemaErrorFormatter?: (errors: FastifySchemaValidationError[], dataVar: string) => Error
+  schemaErrorFormatter?: (errors: FastifySchemaValidationError[], dataVar: string) => Error,
+  /**
+   * listener to error events emitted by client connections
+   */
+  clientErrorHandler?: (error: ConnectionError, socket: Socket) => void
 }
 
 type TrustProxyFunction = (address: string, hop: number) => boolean
@@ -150,7 +164,7 @@ export { FastifyReply } from './types/reply'
 export { FastifyPluginCallback, FastifyPluginAsync, FastifyPluginOptions, FastifyPlugin } from './types/plugin'
 export { FastifyInstance } from './types/instance'
 export { FastifyLoggerOptions, FastifyLoggerInstance, FastifyLogFn, LogLevel } from './types/logger'
-export { FastifyContext } from './types/context'
+export { FastifyContext, FastifyContextConfig } from './types/context'
 export { RouteHandler, RouteHandlerMethod, RouteOptions, RouteShorthandMethod, RouteShorthandOptions, RouteShorthandOptionsWithHandler } from './types/route'
 export * from './types/register'
 export { FastifyBodyParser, FastifyContentTypeParser, AddContentTypeParser, hasContentTypeParser, getDefaultJsonParser, ProtoAction, ConstructorAction } from './types/content-type-parser'

package/fastify.js

@@ -113,10 +113,6 @@ function fastify (options) {
     throw new Error(`ajv.plugins option should be an array, instead got '${typeof ajvOptions.plugins}'`)
   }
 
-  ajvOptions.plugins = ajvOptions.plugins.map(plugin => {
-    return Array.isArray(plugin) ? plugin : [plugin]
-  })
-
   // Instance Fastify components
   const { logger, hasLogger } = createLogger(options)
 
@@ -290,6 +286,9 @@ function fastify (options) {
     initialConfig
   }
 
+  fastify[kReply].prototype.server = fastify
+  fastify[kRequest].prototype.server = fastify
+
   Object.defineProperties(fastify, {
     pluginName: {
       get () {

package/lib/decorate.js

@@ -11,7 +11,8 @@ const {
 const {
   FST_ERR_DEC_ALREADY_PRESENT,
   FST_ERR_DEC_MISSING_DEPENDENCY,
-  FST_ERR_DEC_AFTER_START
+  FST_ERR_DEC_AFTER_START,
+  FST_ERR_DEC_DEPENDENCY_INVALID_TYPE
 } = require('./errors')
 
 const warning = require('./warnings')
@@ -22,6 +23,10 @@ function decorate (instance, name, fn, dependencies) {
   }
 
   if (dependencies) {
+    if (!Array.isArray(dependencies)) {
+      throw new FST_ERR_DEC_DEPENDENCY_INVALID_TYPE(name)
+    }
+
     checkDependencies(instance, dependencies)
   }
 

package/lib/errors.js

@@ -72,6 +72,10 @@ const codes = {
     'FST_ERR_DEC_ALREADY_PRESENT',
     "The decorator '%s' has already been added!"
   ),
+  FST_ERR_DEC_DEPENDENCY_INVALID_TYPE: createError(
+    'FST_ERR_DEC_DEPENDENCY_INVALID_TYPE',
+    "The dependencies of decorator '%s' must be of type Array."
+  ),
   FST_ERR_DEC_MISSING_DEPENDENCY: createError(
     'FST_ERR_DEC_MISSING_DEPENDENCY',
     "The decorator is missing dependency '%s'."

package/lib/pluginOverride.js

@@ -37,8 +37,13 @@ module.exports = function override (old, fn, opts) {
   old[kChildren].push(instance)
   instance.ready = old[kAvvioBoot].bind(instance)
   instance[kChildren] = []
+
   instance[kReply] = Reply.buildReply(instance[kReply])
+  instance[kReply].prototype.server = instance
+
   instance[kRequest] = Request.buildRequest(instance[kRequest])
+  instance[kRequest].prototype.server = instance
+
   instance[kContentTypeParser] = ContentTypeParser.helpers.buildContentTypeParser(instance[kContentTypeParser])
   instance[kHooks] = buildHooks(instance[kHooks])
   instance[kRoutePrefix] = buildRoutePrefix(instance[kRoutePrefix], opts.prefix)

package/lib/reply.js

@@ -101,6 +101,10 @@ Object.defineProperties(Reply.prototype, {
     set (value) {
       this.code(value)
     }
+  },
+  server: {
+    value: null,
+    writable: true
   }
 })
 
@@ -324,12 +328,14 @@ Reply.prototype.then = function (fulfilled, rejected) {
     return
   }
 
-  eos(this.raw, function (err) {
+  eos(this.raw, (err) => {
     // We must not treat ERR_STREAM_PREMATURE_CLOSE as
     // an error because it is created by eos, not by the stream.
     if (err && err.code !== 'ERR_STREAM_PREMATURE_CLOSE') {
       if (rejected) {
         rejected(err)
+      } else {
+        this.log && this.log.warn('unhandled rejection on reply.then')
       }
     } else {
       fulfilled()
@@ -426,6 +432,8 @@ function onSendEnd (reply, payload) {
 
   if (!reply[kReplyHeaders]['content-length']) {
     reply[kReplyHeaders]['content-length'] = '' + Buffer.byteLength(payload)
+  } else if (req.raw.method !== 'HEAD' && reply[kReplyHeaders]['content-length'] !== Buffer.byteLength(payload)) {
+    reply[kReplyHeaders]['content-length'] = '' + Buffer.byteLength(payload)
   }
 
   reply[kReplySent] = true
@@ -478,6 +486,8 @@ function sendStream (payload, res, reply) {
         payload.close(noop)
       } else if (typeof payload.abort === 'function') {
         payload.abort()
+      } else {
+        reply.log.warn('stream payload does not end properly')
       }
     }
   })
@@ -560,11 +570,12 @@ function handleError (reply, error, cb) {
         statusCode: { value: statusCode }
       }))
   } catch (err) {
+    // error is always FST_ERR_SCH_SERIALIZATION_BUILD because this is called from route/compileSchemasForSerialization
     reply.log.error({ err, statusCode: res.statusCode }, 'The serializer for the given status code failed')
     res.statusCode = 500
     payload = serializeError({
       error: statusCodes['500'],
-      message: err.message || '',
+      message: err.message,
       statusCode: 500
     })
   }

package/lib/request.js

@@ -133,6 +133,7 @@ Object.defineProperties(Request.prototype, {
   },
   connection: {
     get () {
+      /* istanbul ignore next */
       if (semver.gte(process.versions.node, '13.0.0')) {
         warning.emit('FSTDEP005')
       }
@@ -163,6 +164,10 @@ Object.defineProperties(Request.prototype, {
     get () {
       return this.raw.headers
     }
+  },
+  server: {
+    value: null,
+    writable: true
   }
 })
 

package/lib/route.js

@@ -326,6 +326,7 @@ function buildRouting (options) {
   // HTTP request entry point, the routing has already been executed
   function routeHandler (req, res, params, context) {
     if (closing === true) {
+      /* istanbul ignore next mac, windows */
       if (req.httpVersionMajor !== 2) {
         res.once('finish', () => req.destroy())
         res.setHeader('Connection', 'close')