fastify 4.21.0 → 4.22.1

package/docs/Guides/Database.md

@@ -308,7 +308,7 @@ async function migrate() {
 
     process.exitCode = 0
   } catch(err) {
-    console.error(error)
+    console.error(err)
     process.exitCode = 1
   }
   

package/docs/Guides/Ecosystem.md

@@ -44,6 +44,9 @@ section.
   Fastify.
 - [`@fastify/diagnostics-channel`](https://github.com/fastify/fastify-diagnostics-channel)
   Plugin to deal with `diagnostics_channel` on Fastify
+- [`@fastify/early-hints`](https://github.com/fastify/fastify-early-hints) Plugin
+  to add HTTP 103 feature based on [RFC
+  8297](https://datatracker.ietf.org/doc/html/rfc8297).
 - [`@fastify/elasticsearch`](https://github.com/fastify/fastify-elasticsearch)
   Plugin to share the same ES client.
 - [`@fastify/env`](https://github.com/fastify/fastify-env) Load and check
@@ -118,6 +121,8 @@ section.
 - [`@fastify/swagger`](https://github.com/fastify/fastify-swagger) Plugin for
   serving Swagger/OpenAPI documentation for Fastify, supporting dynamic
   generation.
+- [`@fastify/throttle`](https://github.com/fastify/fastify-throttle) Plugin for
+- throttling the download speed of a request.
 - [`@fastify/type-provider-json-schema-to-ts`](https://github.com/fastify/fastify-type-provider-json-schema-to-ts)
   Fastify
   [type provider](https://www.fastify.io/docs/latest/Reference/Type-Providers/)
@@ -304,9 +309,6 @@ section.
   object.
 - [`fastify-dynareg`](https://github.com/greguz/fastify-dynareg) Dynamic plugin
   register for Fastify.
-- [`fastify-early-hints`](https://github.com/zekth/fastify-early-hints) Plugin
-  to add HTTP 103 feature based on [RFC
-  8297](https://httpwg.org/specs/rfc8297.html)
 - [`fastify-envalid`](https://github.com/alemagio/fastify-envalid) Fastify
   plugin to integrate [envalid](https://github.com/af/envalid) in your Fastify
   project.
@@ -358,6 +360,8 @@ section.
   Providers.
 - [`fastify-guard`](https://github.com/hsynlms/fastify-guard) A Fastify plugin
   that protects endpoints by checking authenticated user roles and/or scopes.
+- [`fastify-hashids`](https://github.com/andersonjoseph/fastify-hashids) A Fastify
+  plugin to encode/decode IDs using [hashids](https://github.com/niieani/hashids.js).
 - [`fastify-hasura`](https://github.com/ManUtopiK/fastify-hasura) A Fastify
   plugin to have fun with [Hasura](https://github.com/hasura/graphql-engine).
 - [`fastify-healthcheck`](https://github.com/smartiniOnGitHub/fastify-healthcheck)
@@ -518,6 +522,10 @@ middlewares into Fastify plugins
 - [`fastify-qs`](https://github.com/vanodevium/fastify-qs) A plugin for Fastify
   that adds support for parsing URL query parameters with
   [qs](https://github.com/ljharb/qs).
+- [`fastify-rabbitmq`](https://github.com/Bugs5382/fastify-rabbitmq) Fastify
+  RabbitMQ plugin that uses
+  [node-amqp-connection-manager](https://github.com/jwalton/node-amqp-connection-manager)
+  plugin as a wrapper.
 - [`fastify-racing`](https://github.com/metcoder95/fastify-racing) Fastify's
   plugin that adds support to handle an aborted request asynchronous.
 - [`fastify-ravendb`](https://github.com/nearform/fastify-ravendb) RavenDB

package/docs/Guides/Getting-Started.md

@@ -277,7 +277,7 @@ async function dbConnector (fastify, options) {
 
 // Wrapping a plugin function with fastify-plugin exposes the decorators
 // and hooks, declared inside the plugin to the parent scope.
-module.exports = fastifyPlugin(dbConnector)
+export default fastifyPlugin(dbConnector)
 
 ```
 

package/docs/Guides/Serverless.md

@@ -487,10 +487,10 @@ with your GitHub account.
 Create your first application and a static workspace: be careful to download the
 API key as an env file, e.g. `yourworkspace.txt`.
 
-Then, you can easily deploy your application with the following commmand:
+Then, you can easily deploy your application with the following command:
 
 ```bash
-platformatic deploy --keys `yuourworkspace.txt`
+platformatic deploy --keys `yourworkspace.txt`
 ```
 
 Check out the [Full Guide](https://blog.platformatic.dev/how-to-migrate-a-fastify-app-to-platformatic-service)

package/docs/Reference/Errors.md

@@ -510,7 +510,7 @@ Impossible to load plugin because the parent (mapped directly from `avvio`)
 #### FST_ERR_PLUGIN_TIMEOUT
 <a id="FST_ERR_PLUGIN_TIMEOUT"></a>
 
-Plugin did not start in time. Default timeout (in millis): `10000`
+Plugin did not start in time. Default timeout (in milliseconds): `10000`
 
 #### FST_ERR_PLUGIN_NOT_PRESENT_IN_INSTANCE
 <a id="FST_ERR_PLUGIN_NOT_PRESENT_IN_INSTANCE"></a>

package/docs/Reference/HTTP2.md

@@ -32,7 +32,8 @@ fastify.get('/', function (request, reply) {
 fastify.listen({ port: 3000 })
 ```
 
-ALPN negotiation allows support for both HTTPS and HTTP/2 over the same socket.
+[ALPN negotiation](https://datatracker.ietf.org/doc/html/rfc7301) allows
+support for both HTTPS and HTTP/2 over the same socket.
 Node core `req` and `res` objects can be either
 [HTTP/1](https://nodejs.org/api/http.html) or
 [HTTP/2](https://nodejs.org/api/http2.html). _Fastify_ supports this out of the

package/docs/Reference/Hooks.md

@@ -80,7 +80,7 @@ hooks, and a stream with the current request payload.
 If it returns a value (via `return` or via the callback function), it must
 return a stream.
 
-For instance, you can uncompress the request body:
+For instance, you can decompress the request body:
 
 ```js
 fastify.addHook('preParsing', (request, reply, payload, done) => {
@@ -106,6 +106,9 @@ returned stream. This property is used to correctly match the request payload
 with the `Content-Length` header value. Ideally, this property should be updated
 on each received chunk.
 
+**Notice:** The size of the returned stream is checked to not exceed the limit 
+set in [`bodyLimit`](./Server.md#bodylimit) option.
+
 ### preValidation
 
 If you are using the `preValidation` hook, you can change the payload before it

package/docs/Reference/Server.md

@@ -282,6 +282,10 @@ attacks](https://www.owasp.org/index.php/Regular_expression_Denial_of_Service_-_
 + Default: `1048576` (1MiB)
 
 Defines the maximum payload, in bytes, the server is allowed to accept.
+The default body reader sends [`FST_ERR_CTP_BODY_TOO_LARGE`](./Errors.md#fst_err_ctp_body_too_large)
+reply, if the size of the body exceeds this limit.
+If [`preParsing` hook](./Hooks.md#preparsing) is provided, this limit is applied
+to the size of the stream the hook returns (i.e. the size of "decoded" body).
 
 ### `onProtoPoisoning`
 <a id="factory-on-proto-poisoning"></a>
@@ -476,7 +480,7 @@ is not equal to `/Foo`.
 When `false` then routes are case-insensitive.
 
 Please note that setting this option to `false` goes against
-[RFC3986](https://tools.ietf.org/html/rfc3986#section-6.2.2.1).
+[RFC3986](https://datatracker.ietf.org/doc/html/rfc3986#section-6.2.2.1).
 
 By setting `caseSensitive` to `false`, all paths will be matched as lowercase,
 but the route parameters or wildcards will maintain their original letter

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

@@ -397,7 +397,7 @@ configuration](https://github.com/fastify/ajv-compiler#ajv-configuration) is:
 {
   coerceTypes: 'array', // change data type of data to match type keyword
   useDefaults: true, // replace missing properties and items with the values from corresponding default keyword
-  removeAdditional: true, // remove additional properties
+  removeAdditional: true, // remove additional properties if additionalProperties is set to false, see: https://ajv.js.org/guide/modifying-data.html#removing-additional-properties
   uriResolver: require('fast-uri'),
   addUsedSchema: false,
   // Explicitly set allErrors to `false`.

package/fastify.d.ts

@@ -20,15 +20,15 @@ import { FastifyRegister, FastifyRegisterOptions, RegisterOptions } from './type
 import { FastifyReply } from './types/reply'
 import { FastifyRequest, RequestGenericInterface } from './types/request'
 import { RouteHandler, RouteHandlerMethod, RouteOptions, RouteShorthandMethod, RouteShorthandOptions, RouteShorthandOptionsWithHandler, RouteGenericInterface } from './types/route'
-import { FastifySchema, FastifySchemaCompiler, SchemaErrorDataVar, SchemaErrorFormatter } from './types/schema'
+import { FastifySchema, FastifySchemaCompiler, FastifySchemaValidationError, SchemaErrorDataVar, SchemaErrorFormatter } from './types/schema'
 import { FastifyServerFactory, FastifyServerFactoryHandler } from './types/serverFactory'
 import { FastifyTypeProvider, FastifyTypeProviderDefault } from './types/type-provider'
 import { HTTPMethods, RawServerBase, RawRequestDefaultExpression, RawReplyDefaultExpression, RawServerDefault, ContextConfigDefault, RequestBodyDefault, RequestQuerystringDefault, RequestParamsDefault, RequestHeadersDefault } from './types/utils'
 
 declare module '@fastify/error' {
   interface FastifyError {
-    validation?: fastify.ValidationResult[];
     validationContext?: SchemaErrorDataVar;
+    validation?: FastifySchemaValidationError[];
   }
 }
 
@@ -162,13 +162,10 @@ declare namespace fastify {
     clientErrorHandler?: (error: ConnectionError, socket: Socket) => void,
   }
 
-  export interface ValidationResult {
-    keyword: string;
-    instancePath: string;
-    schemaPath: string;
-    params: Record<string, string | string[]>;
-    message?: string;
-  }
+  /**
+   * @deprecated use {@link FastifySchemaValidationError}
+   */
+  export type ValidationResult = FastifySchemaValidationError;
 
   /* Export additional types */
   export type {
@@ -241,4 +238,4 @@ declare function fastify<
 
 // CJS export
 // const fastify = require('fastify')
-export = fastify
+export = fastify

package/fastify.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const VERSION = '4.21.0'
+const VERSION = '4.22.1'
 
 const Avvio = require('avvio')
 const http = require('http')
@@ -241,7 +241,7 @@ function fastify (options) {
     [kReply]: Reply.buildReply(Reply),
     [kRequest]: Request.buildRequest(Request, options.trustProxy),
     [kFourOhFour]: fourOhFour,
-    [pluginUtils.registeredPlugins]: [],
+    [pluginUtils.kRegisteredPlugins]: [],
     [kPluginNameChain]: ['fastify'],
     [kAvvioBoot]: null,
     // routing method
@@ -312,7 +312,7 @@ function fastify (options) {
     close: null,
     printPlugins: null,
     hasPlugin: function (name) {
-      return this[kPluginNameChain].includes(name)
+      return this[pluginUtils.kRegisteredPlugins].includes(name) || this[kPluginNameChain].includes(name)
     },
     // http server
     listen,

package/lib/contentTypeParser.js

@@ -102,7 +102,7 @@ ContentTypeParser.prototype.getParser = function (contentType) {
   const parsed = safeParseContentType(contentType)
 
   // dummyContentType always the same object
-  // we can use === for the comparsion and return early
+  // we can use === for the comparison and return early
   if (parsed === defaultContentType) {
     return this.customParsers.get('')
   }
@@ -226,8 +226,11 @@ function rawBody (request, reply, options, parser, done) {
 
   function onData (chunk) {
     receivedLength += chunk.length
-
-    if ((payload.receivedEncodedLength || receivedLength) > limit) {
+    const { receivedEncodedLength = 0 } = payload
+    // The resulting body length must not exceed bodyLimit (see "zip bomb").
+    // The case when encoded length is larger than received length is rather theoretical,
+    // unless the stream returned by preParsing hook is broken and reports wrong value.
+    if (receivedLength > limit || receivedEncodedLength > limit) {
       payload.removeListener('data', onData)
       payload.removeListener('end', onEnd)
       payload.removeListener('error', onEnd)
@@ -396,7 +399,7 @@ function compareRegExpContentType (contentType, essenceMIMEType, regexp) {
 function ParserListItem (contentType) {
   this.name = contentType
   // we pre-calculate all the needed information
-  // before content-type comparsion
+  // before content-type comparison
   const parsed = safeParseContentType(contentType)
   this.isEssence = contentType.indexOf(';') === -1
   // we should not allow empty string for parser list item

package/lib/error-serializer.js

@@ -16,41 +16,43 @@
 ) {
 
     
-    function anonymous0 (input) {
-      // #
   
+    // #
+    function anonymous0 (input) {
       const obj = (input && typeof input.toJSON === 'function')
     ? input.toJSON()
     : input
   
-      let json = '{'
-      let addComma = false
+      
+    let addComma = false
+    let json = '{'
+  
+        if (obj["statusCode"] !== undefined) {
+          !addComma && (addComma = true) || (json += ',')
+          json += "\"statusCode\":"
+          json += serializer.asNumber(obj["statusCode"])
+        }
+        
+        if (obj["code"] !== undefined) {
+          !addComma && (addComma = true) || (json += ',')
+          json += "\"code\":"
+          json += serializer.asString(obj["code"])
+        }
+        
+        if (obj["error"] !== undefined) {
+          !addComma && (addComma = true) || (json += ',')
+          json += "\"error\":"
+          json += serializer.asString(obj["error"])
+        }
+        
+        if (obj["message"] !== undefined) {
+          !addComma && (addComma = true) || (json += ',')
+          json += "\"message\":"
+          json += serializer.asString(obj["message"])
+        }
+        
+  return json + '}'
   
-      if (obj["statusCode"] !== undefined) {
-        !addComma && (addComma = true) || (json += ',')
-        json += "\"statusCode\":"
-      json += serializer.asNumber(obj["statusCode"])
-      }
-    
-      if (obj["code"] !== undefined) {
-        !addComma && (addComma = true) || (json += ',')
-        json += "\"code\":"
-      json += serializer.asString(obj["code"])
-      }
-    
-      if (obj["error"] !== undefined) {
-        !addComma && (addComma = true) || (json += ',')
-        json += "\"error\":"
-      json += serializer.asString(obj["error"])
-      }
-    
-      if (obj["message"] !== undefined) {
-        !addComma && (addComma = true) || (json += ',')
-        json += "\"message\":"
-      json += serializer.asString(obj["message"])
-      }
-    
-      return json + '}'
     }
   
     const main = anonymous0

package/lib/handleRequest.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const { validate: validateSchema } = require('./validation')
-const { onPreValidationHookRunner, onPreHandlerHookRunner } = require('./hooks')
+const { preValidationHookRunner, preHandlerHookRunner } = require('./hooks')
 const wrapThenable = require('./wrapThenable')
 const {
   kReplyIsError,
@@ -65,7 +65,7 @@ function handleRequest (err, request, reply) {
 function handler (request, reply) {
   try {
     if (request[kRouteContext].preValidation !== null) {
-      onPreValidationHookRunner(
+      preValidationHookRunner(
         request[kRouteContext].preValidation,
         request,
         reply,
@@ -111,7 +111,7 @@ function validationCompleted (request, reply, validationErr) {
 
   // preHandler hook
   if (request[kRouteContext].preHandler !== null) {
-    onPreHandlerHookRunner(
+    preHandlerHookRunner(
       request[kRouteContext].preHandler,
       request,
       reply,

package/lib/hooks.js

@@ -217,8 +217,8 @@ function onResponseHookIterator (fn, request, reply, next) {
 }
 
 const onResponseHookRunner = hookRunnerGenerator(onResponseHookIterator)
-const onPreValidationHookRunner = hookRunnerGenerator(hookIterator)
-const onPreHandlerHookRunner = hookRunnerGenerator(hookIterator)
+const preValidationHookRunner = hookRunnerGenerator(hookIterator)
+const preHandlerHookRunner = hookRunnerGenerator(hookIterator)
 const onTimeoutHookRunner = hookRunnerGenerator(hookIterator)
 const onRequestHookRunner = hookRunnerGenerator(hookIterator)
 
@@ -267,6 +267,8 @@ function onSendHookRunner (functions, request, reply, payload, cb) {
   next()
 }
 
+const preSerializationHookRunner = onSendHookRunner
+
 function preParsingHookRunner (functions, request, reply, cb) {
   let i = 0
 
@@ -360,11 +362,12 @@ module.exports = {
   preParsingHookRunner,
   onResponseHookRunner,
   onSendHookRunner,
+  preSerializationHookRunner,
   onRequestAbortHookRunner,
   hookIterator,
   hookRunnerApplication,
-  onPreHandlerHookRunner,
-  onPreValidationHookRunner,
+  preHandlerHookRunner,
+  preValidationHookRunner,
   onRequestHookRunner,
   onTimeoutHookRunner,
   lifecycleHooks,

package/lib/logger.js

@@ -144,7 +144,7 @@ function createChildLogger (context, logger, req, reqId, loggerOpts) {
   }
   const child = context.childLoggerFactory.call(context.server, logger, loggerBindings, loggerOpts || {}, req)
 
-  // Optimisation: bypass validation if the factory is our own default factory
+  // Optimization: bypass validation if the factory is our own default factory
   if (context.childLoggerFactory !== defaultChildLoggerFactory) {
     validateLogger(child, true) // throw if the child is not a valid logger
   }

package/lib/pluginOverride.js

@@ -27,9 +27,10 @@ const pluginUtils = require('./pluginUtils')
 module.exports = function override (old, fn, opts) {
   const shouldSkipOverride = pluginUtils.registerPlugin.call(old, fn)
 
+  const fnName = pluginUtils.getPluginName(fn) || pluginUtils.getFuncPreview(fn)
   if (shouldSkipOverride) {
     // after every plugin registration we will enter a new name
-    old[kPluginNameChain].push(pluginUtils.getDisplayName(fn))
+    old[kPluginNameChain].push(fnName)
     return old
   }
 
@@ -48,8 +49,14 @@ module.exports = function override (old, fn, opts) {
   instance[kSchemaController] = SchemaController.buildSchemaController(old[kSchemaController])
   instance.getSchema = instance[kSchemaController].getSchema.bind(instance[kSchemaController])
   instance.getSchemas = instance[kSchemaController].getSchemas.bind(instance[kSchemaController])
-  instance[pluginUtils.registeredPlugins] = Object.create(instance[pluginUtils.registeredPlugins])
-  instance[kPluginNameChain] = [pluginUtils.getPluginName(fn) || pluginUtils.getFuncPreview(fn)]
+
+  // Track the registered and loaded plugins since the root instance.
+  // It does not track the current encapsulated plugin.
+  instance[pluginUtils.kRegisteredPlugins] = Object.create(instance[pluginUtils.kRegisteredPlugins])
+
+  // Track the plugin chain since the root instance.
+  // When an non-encapsulated plugin is added, the chain will be updated.
+  instance[kPluginNameChain] = [fnName]
 
   if (instance[kLogSerializers] || opts.logSerializers) {
     instance[kLogSerializers] = Object.assign(Object.create(instance[kLogSerializers]), opts.logSerializers)

package/lib/pluginUtils.js

@@ -2,7 +2,7 @@
 
 const semver = require('semver')
 const assert = require('assert')
-const registeredPlugins = Symbol.for('registered-plugin')
+const kRegisteredPlugins = Symbol.for('registered-plugin')
 const {
   kTestInternals
 } = require('./symbols.js')
@@ -25,12 +25,15 @@ function getPluginName (func) {
   // let's see if this is a file, and in that case use that
   // this is common for plugins
   const cache = require.cache
-  const keys = Object.keys(cache)
-
-  for (let i = 0; i < keys.length; i++) {
-    const key = keys[i]
-    if (cache[key].exports === func) {
-      return key
+  // cache is undefined inside SEA
+  if (cache) {
+    const keys = Object.keys(cache)
+
+    for (let i = 0; i < keys.length; i++) {
+      const key = keys[i]
+      if (cache[key].exports === func) {
+        return key
+      }
     }
   }
 
@@ -65,7 +68,7 @@ function checkDependencies (fn) {
 
   dependencies.forEach(dependency => {
     assert(
-      this[registeredPlugins].indexOf(dependency) > -1,
+      this[kRegisteredPlugins].indexOf(dependency) > -1,
       `The dependency '${dependency}' of plugin '${meta.name}' is not registered`
     )
   })
@@ -128,7 +131,7 @@ function registerPluginName (fn) {
 
   const name = meta.name
   if (!name) return
-  this[registeredPlugins].push(name)
+  this[kRegisteredPlugins].push(name)
 }
 
 function registerPlugin (fn) {
@@ -142,7 +145,7 @@ function registerPlugin (fn) {
 module.exports = {
   getPluginName,
   getFuncPreview,
-  registeredPlugins,
+  kRegisteredPlugins,
   getDisplayName,
   registerPlugin
 }

package/lib/reply.js

@@ -23,7 +23,12 @@ const {
   kOptions,
   kRouteContext
 } = require('./symbols.js')
-const { onSendHookRunner, onResponseHookRunner, onPreHandlerHookRunner } = require('./hooks')
+const {
+  onSendHookRunner,
+  onResponseHookRunner,
+  preHandlerHookRunner,
+  preSerializationHookRunner
+} = require('./hooks')
 
 const internals = require('./handleRequest')[Symbol.for('internals')]
 const loggerUtils = require('./logger')
@@ -166,7 +171,7 @@ Reply.prototype.send = function (payload) {
 
   if (this[kReplySerializer] !== null) {
     if (typeof payload !== 'string') {
-      preserializeHook(this, payload)
+      preSerializationHook(this, payload)
       return this
     } else {
       payload = this[kReplySerializer](payload)
@@ -189,7 +194,7 @@ Reply.prototype.send = function (payload) {
       }
     }
     if (typeof payload !== 'string') {
-      preserializeHook(this, payload)
+      preSerializationHook(this, payload)
       return this
     }
   }
@@ -233,7 +238,7 @@ Reply.prototype.header = function (key, value = '') {
   key = key.toLowerCase()
 
   if (this[kReplyHeaders][key] && key === 'set-cookie') {
-    // https://tools.ietf.org/html/rfc7230#section-3.2.2
+    // https://datatracker.ietf.org/doc/html/rfc7230#section-3.2.2
     if (typeof this[kReplyHeaders][key] === 'string') {
       this[kReplyHeaders][key] = [this[kReplyHeaders][key]]
     }
@@ -262,7 +267,7 @@ Reply.prototype.headers = function (headers) {
 }
 
 // https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Trailer#directives
-// https://httpwg.org/specs/rfc7230.html#chunked.trailer.part
+// https://datatracker.ietf.org/doc/html/rfc7230.html#chunked.trailer.part
 const INVALID_TRAILERS = new Set([
   'transfer-encoding',
   'content-length',
@@ -476,21 +481,21 @@ Reply.prototype.then = function (fulfilled, rejected) {
   })
 }
 
-function preserializeHook (reply, payload) {
+function preSerializationHook (reply, payload) {
   if (reply[kRouteContext].preSerialization !== null) {
-    onSendHookRunner(
+    preSerializationHookRunner(
       reply[kRouteContext].preSerialization,
       reply.request,
       reply,
       payload,
-      preserializeHookEnd
+      preSerializationHookEnd
     )
   } else {
-    preserializeHookEnd(null, reply.request, reply, payload)
+    preSerializationHookEnd(null, reply.request, reply, payload)
   }
 }
 
-function preserializeHookEnd (err, request, reply, payload) {
+function preSerializationHookEnd (err, request, reply, payload) {
   if (err != null) {
     onErrorHook(reply, err)
     return
@@ -505,7 +510,7 @@ function preserializeHookEnd (err, request, reply, payload) {
       payload = serialize(reply[kRouteContext], payload, reply.raw.statusCode, reply[kReplyHeaders]['content-type'])
     }
   } catch (e) {
-    wrapSeralizationError(e, reply)
+    wrapSerializationError(e, reply)
     onErrorHook(reply, e)
     return
   }
@@ -513,7 +518,7 @@ function preserializeHookEnd (err, request, reply, payload) {
   onSendHook(reply, payload)
 }
 
-function wrapSeralizationError (error, reply) {
+function wrapSerializationError (error, reply) {
   error.serialization = reply[kRouteContext].config
 }
 
@@ -571,7 +576,7 @@ function onSendEnd (reply, payload) {
   }
 
   if (payload === undefined || payload === null) {
-    // according to https://tools.ietf.org/html/rfc7230#section-3.3.2
+    // according to https://datatracker.ietf.org/doc/html/rfc7230#section-3.3.2
     // we cannot send a content-length for 304 and 204, and all status code
     // < 200
     // A sender MUST NOT send a Content-Length header field in any message
@@ -846,7 +851,7 @@ function notFound (reply) {
 
   // preHandler hook
   if (reply[kRouteContext].preHandler !== null) {
-    onPreHandlerHookRunner(
+    preHandlerHookRunner(
       reply[kRouteContext].preHandler,
       reply.request,
       reply,

package/lib/request.js

@@ -51,8 +51,8 @@ function getTrustProxyFn (tp) {
   }
   if (typeof tp === 'string') {
     // Support comma-separated tps
-    const vals = tp.split(',').map(it => it.trim())
-    return proxyAddr.compile(vals)
+    const values = tp.split(',').map(it => it.trim())
+    return proxyAddr.compile(values)
   }
   return proxyAddr.compile(tp)
 }

package/lib/schemas.js

@@ -23,7 +23,7 @@ Schemas.prototype.add = function (inputSchema) {
     : inputSchema
   )
 
-  // devs can add schemas without $id, but with $def instead
+  // developers can add schemas without $id, but with $def instead
   const id = schema.$id
   if (!id) {
     throw new FST_ERR_SCH_MISSING_ID()

package/lib/server.js

@@ -132,7 +132,7 @@ function multipleBindings (mainServer, httpHandler, serverOpts, listenOptions, o
     const isMainServerListening = mainServer.listening && serverOpts.serverFactory
 
     let binding = 0
-    let binded = 0
+    let bound = 0
     if (!isMainServerListening) {
       const primaryAddress = mainServer.address()
       for (const adr of addresses) {
@@ -142,7 +142,7 @@ function multipleBindings (mainServer, httpHandler, serverOpts, listenOptions, o
             host: adr.address,
             port: primaryAddress.port,
             cb: (_ignoreErr) => {
-              binded++
+              bound++
 
               /* istanbul ignore next: the else won't be taken unless listening fails */
               if (!_ignoreErr) {
@@ -172,7 +172,7 @@ function multipleBindings (mainServer, httpHandler, serverOpts, listenOptions, o
                 })
               }
 
-              if (binded === binding) {
+              if (bound === binding) {
                 // regardless of the error, we are done
                 onListen()
               }
@@ -200,7 +200,7 @@ function multipleBindings (mainServer, httpHandler, serverOpts, listenOptions, o
     // to the secondary servers. It is valid only when the user is
     // listening on localhost
     const originUnref = mainServer.unref
-    /* istanbul ignore next */
+    /* c8 ignore next 4 */
     mainServer.unref = function () {
       originUnref.call(mainServer)
       mainServer.emit('unref')