fastify 5.11.2 → 6.0.0-alpha.0

package/docs/Reference/Server.md

@@ -22,11 +22,9 @@ describes the properties available in that options object.
   - [`onConstructorPoisoning`](#onconstructorpoisoning)
   - [`logger`](#logger)
   - [`loggerInstance`](#loggerinstance)
-  - [`disableRequestLogging`](#disablerequestlogging)
   - [`logController`](#logcontroller)
   - [`serverFactory`](#serverfactory)
   - [`requestIdHeader`](#requestidheader)
-  - [`requestIdLogLabel`](#requestidloglabel)
   - [`genReqId`](#genreqid)
   - [`trustProxy`](#trustproxy)
   - [`pluginTimeout`](#plugintimeout)
@@ -387,71 +385,6 @@ Pino interface by having the following methods: `info`, `error`, `debug`,
   const fastify = require('fastify')({ loggerInstance: customLogger });
   ```
 
-### `disableRequestLogging`
-<a id="factory-disable-request-logging"></a>
-
-> **Deprecated:** Use the [`logController`](#factory-log-controller)
-> option with `disableRequestLogging` or `isLogDisabled` override instead.
-> This top-level option will be removed in `fastify@6`.
-
-+ Default: `false`
-
-When logging is enabled, Fastify will issue an `info` level log
-message when a request is received and when the response for that request has
-been sent. By setting this option to `true`, these log messages will be
-disabled. This allows for more flexible request start and end logging by
-attaching custom `onRequest` and `onResponse` hooks.
-
-This option can also be a function that receives the Fastify request object
-and returns a boolean. This allows for conditional request logging based on the
-request properties (e.g., URL, headers, decorations).
-
-```js
-const { LogController } = require('fastify')
-
-// Deprecated
-const fastify = require('fastify')({
-  logger: true,
-  disableRequestLogging: (request) => {
-    return request.url === '/health' || request.url === '/ready'
-  }
-})
-
-// Recommended: use logController instead
-const fastify = require('fastify')({
-  logger: true,
-  logController: new LogController({
-    disableRequestLogging: (request) => {
-      return request.url === '/health' || request.url === '/ready'
-    }
-  })
-})
-```
-
-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`
-on error management
-- the info log written by the `fourOhFour` handler when a
-non existent route is requested
-
-Other log messages emitted by Fastify will stay enabled,
-like deprecation warnings and messages
-emitted when requests are received while the server is closing.
-
-```js
-// Examples of hooks to replicate the disabled functionality.
-fastify.addHook('onRequest', (req, reply, done) => {
-  req.log.info({ url: req.raw.url, id: req.id }, 'received request')
-  done()
-})
-
-fastify.addHook('onResponse', (req, reply, done) => {
-  req.log.info({ url: req.raw.originalUrl, statusCode: reply.raw.statusCode }, 'request completed')
-  done()
-})
-```
-
 ### `logController`
 <a id="factory-log-controller"></a>
 
@@ -593,17 +526,6 @@ const fastify = require('fastify')({
 > value of their choosing.
 > No validation is performed on `requestIdHeader`.
 
-### `requestIdLogLabel`
-<a id="factory-request-id-log-label"></a>
-
-> **Deprecated:** Use the [`logController`](#factory-log-controller)
-> option with `requestIdLogLabel` instead. This top-level option will be
-> removed in `fastify@6`.
-
-+ Default: `'reqId'`
-
-Defines the label used for the request identifier when logging the request.
-
 ### `genReqId`
 <a id="factory-gen-request-id"></a>
 
@@ -1227,16 +1149,15 @@ fastify.get('/dev', async (request, reply) => {
 ### `allowErrorHandlerOverride`
 <a id="allow-error-handler-override"></a>
 
-* **Default:** `true`
-
-> ⚠ Warning:
-> This option will be set to `false` by default
-> in the next major release.
+* **Default:** `false`
 
 When set to `false`, it prevents `setErrorHandler` from being called
 multiple times within the same scope, ensuring that the previous error
 handler is not unintentionally overridden.
 
+Set this option to `true` to allow an error handler to be overridden within
+the same scope.
+
 #### Example of incorrect usage:
 
 ```js
@@ -1732,7 +1653,8 @@ fastify.mkcol('/', (req, reply) => {
 })
 ```
 
-Calling `addHttpMethod` for an existing method overrides its body behavior.
+Calling `addHttpMethod` for an existing method requires `overrideExisting: true`
+and overrides its body behavior.
 
 ```js
 fastify.addHttpMethod('GET', {
@@ -1741,8 +1663,8 @@ fastify.addHttpMethod('GET', {
 })
 ```
 
-In Fastify v5, omitting `overrideExisting: true` emits `FSTDEP025`; in Fastify
-v6, it will throw an error.
+Omitting `overrideExisting: true` when the method already exists throws
+`FST_ERR_ROUTE_METHOD_ALREADY_SUPPORTED`.
 
 #### addSchema
 <a id="add-schema"></a>
@@ -2446,12 +2368,10 @@ The properties that can currently be exposed are:
 - http2
 - https (it will return `false`/`true` or `{ allowHTTP1: true/false }` if
   explicitly passed)
-- disableRequestLogging
 - onProtoPoisoning
 - onConstructorPoisoning
 - pluginTimeout
 - requestIdHeader
-- requestIdLogLabel
 - http2SessionTimeout
 - routerOptions
   - allowUnsafeRegex

package/docs/Reference/TypeScript.md

@@ -1334,14 +1334,6 @@ Interface method definition used within the
 Interface method definition used within the
 [`fastify.register()`][FastifyRegister] method.
 
-##### fastify.FastifyPlugin< [Options][FastifyPluginOptions]>
-[src](https://github.com/fastify/fastify/blob/main/types/plugin.d.ts#L29)
-
-Interface method definition used within the
-[`fastify.register()`][FastifyRegister] method. Document deprecated in favor of
-`FastifyPluginCallback` and `FastifyPluginAsync` since general `FastifyPlugin`
-doesn't properly infer types for async functions.
-
 ##### fastify.FastifyPluginOptions
 [src](https://github.com/fastify/fastify/blob/main/types/plugin.d.ts#L31)
 
@@ -1358,8 +1350,6 @@ extends FastifyPluginOptions`) so they can be passed to the register method.
 [src](https://github.com/fastify/fastify/blob/main/types/register.d.ts#L9)
 ##### fastify.FastifyRegister(plugin: [FastifyPluginAsync][FastifyPluginAsync], opts: [FastifyRegisterOptions][FastifyRegisterOptions])
 [src](https://github.com/fastify/fastify/blob/main/types/register.d.ts#L9)
-##### fastify.FastifyRegister(plugin: [FastifyPlugin][FastifyPlugin], opts: [FastifyRegisterOptions][FastifyRegisterOptions])
-[src](https://github.com/fastify/fastify/blob/main/types/register.d.ts#L9)
 
 This type interface specifies the type for the
 [`fastify.register()`](./Server.md#register) method. The type interface returns
@@ -1368,8 +1358,7 @@ a function signature with an underlying generic `Options` which is defaulted to
 FastifyPlugin parameter when calling this function so there is no need to
 specify the underlying generic. The options parameter is the intersection of the
 plugin's options and two additional optional properties: `prefix: string` and
-`logLevel`: [LogLevel][LogLevel]. `FastifyPlugin` is deprecated use
-`FastifyPluginCallback` and `FastifyPluginAsync` instead.
+`logLevel`: [LogLevel][LogLevel].
 
 Below is an example of the options inference in action:
 
@@ -1427,39 +1416,6 @@ Union type of: `'info' | 'error' | 'debug' | 'fatal' | 'warn' | 'trace'`
 
 ---
 
-#### Context
-
-The context type definition is similar to the other highly dynamic pieces of the
-type system. Route context is available in the route handler method.
-
-##### fastify.FastifyRequestContext
-
-[src](https://github.com/fastify/fastify/blob/main/types/context.d.ts#L11)
-
-An interface with a single required property `config` that is set by default to
-`unknown`. Can be specified either using a generic or an overload.
-
-This type definition is potentially incomplete. If you are using it and can
-provide more details on how to improve the definition, we strongly encourage you
-to open an issue in the main
-[fastify/fastify](https://github.com/fastify/fastify) repository. Thank you in
-advanced!
-
-##### fastify.FastifyReplyContext
-
-[src](https://github.com/fastify/fastify/blob/main/types/context.d.ts#L11)
-
-An interface with a single required property `config` that is set by default to
-`unknown`. Can be specified either using a generic or an overload.
-
-This type definition is potentially incomplete. If you are using it and can
-provide more details on how to improve the definition, we strongly encourage you
-to open an issue in the main
-[fastify/fastify](https://github.com/fastify/fastify) repository. Thank you in
-advanced!
-
----
-
 #### Routing
 
 One of the core principles in Fastify is its routing capabilities. Most of the
@@ -1559,9 +1515,9 @@ FastifyError is a custom error object that includes status code and validation
 results.
 
 It extends the Node.js `Error` type, and adds two additional, optional
-properties: `statusCode: number` and `validation: ValidationResult[]`.
+properties: `statusCode: number` and `validation: FastifySchemaValidationError[]`.
 
-##### fastify.ValidationResult
+##### fastify.FastifySchemaValidationError
 
 [src](https://github.com/fastify/fastify/blob/main/fastify.d.ts#L184)
 
@@ -1713,8 +1669,6 @@ database.
 [FastifyInstance]: #fastifyfastifyinstance
 [FastifyLoggerOptions]: #fastifyfastifyloggeroptions
 [ContextConfigGeneric]: #ContextConfigGeneric
-[FastifyPlugin]:
-    #fastifyfastifypluginoptions-rawserver-rawrequest-requestgeneric
 [FastifyPluginCallback]: #fastifyfastifyplugincallbackoptions
 [FastifyPluginAsync]: #fastifyfastifypluginasyncoptions
 [FastifyPluginOptions]: #fastifyfastifypluginoptions

package/docs/Reference/Warnings.md

@@ -7,12 +7,7 @@
   - [Fastify Warning Codes](#fastify-warning-codes)
     - [FSTWRN001](#FSTWRN001)
     - [FSTWRN003](#FSTWRN003)
-    - [FSTWRN004](#FSTWRN004)
   - [Fastify Deprecation Codes](#fastify-deprecation-codes)
-    - [FSTDEP022](#FSTDEP022)
-    - [FSTDEP023](#FSTDEP023)
-    - [FSTDEP024](#FSTDEP024)
-    - [FSTDEP025](#FSTDEP025)
 
 ## Warnings
 
@@ -34,7 +29,7 @@ one of the following methods:
 - Set the `NODE_NO_WARNINGS` environment variable to `1`
 - Pass the `--no-warnings` flag to the node process
 - Set `no-warnings` in the `NODE_OPTIONS` environment variable
-- Pass `--disable-warning=FSTWRN004` to disable a specific warning
+- Pass `--disable-warning=FSTWRN003` to disable a specific warning
 
 For more information on disabling warnings, see [Node's documentation](https://nodejs.org/api/cli.html).
 
@@ -46,7 +41,7 @@ Disabling warnings is not recommended and may cause unexpected behavior.
 | ---- | ----------- | ------------ | ---------- |
 | <a id="FSTWRN001">FSTWRN001</a> | The specified schema for a route is missing. This may indicate the schema is not well specified. | Check the schema for the route. | [#4647](https://github.com/fastify/fastify/pull/4647) |
 | <a id="FSTWRN003">FSTWRN003</a> | The `%s` plugin mixes async and callback styles, which may lead to unhandled rejections. | Do not mix async and callback style. | [#6011](https://github.com/fastify/fastify/pull/6011) |
-| <a id="FSTWRN004">FSTWRN004</a> | An `errorHandler` is being overridden in the same scope, which can lead to subtle bugs. | Avoid calling `setErrorHandler` more than once in the same scope. For more information, see [Server documentation](https://fastify.dev/docs/latest/Reference/Server/#allowerrorhandleroverride). | [#6104](https://github.com/fastify/fastify/pull/6104) |
+
 ### Fastify Deprecation Codes
 
 Deprecation codes are supported by the Node.js CLI options:
@@ -55,10 +50,8 @@ Deprecation codes are supported by the Node.js CLI options:
 - [--throw-deprecation](https://nodejs.org/api/cli.html#throw-deprecation)
 - [--trace-deprecation](https://nodejs.org/api/cli.html#trace-deprecation)
 
+There are currently no active deprecation codes because the APIs deprecated in
+Fastify v5 have been removed in Fastify v6.
 
 | Code | Description | How to solve | Discussion |
 | ---- | ----------- | ------------ | ---------- |
-| <a id="FSTDEP022">FSTDEP022</a> | You are trying to access the deprecated router options on top option properties. | Use `options.routerOptions`. | [#5985](https://github.com/fastify/fastify/pull/5985)
-| <a id="FSTDEP023">FSTDEP023</a> | `disableRequestLogging` top-level option is deprecated. | Pass a `LogController` instance via the `logController` option with `disableRequestLogging` in its constructor instead. |
-| <a id="FSTDEP024">FSTDEP024</a> | `requestIdLogLabel` top-level option is deprecated. | Pass a `LogController` instance via the `logController` option with `requestIdLogLabel` in its constructor instead. |
-| <a id="FSTDEP025">FSTDEP025</a> | Calling `addHttpMethod` for an existing HTTP method without `{ overrideExisting: true }` is deprecated. | Pass `{ overrideExisting: true }` to make the override explicit. | [#6879](https://github.com/fastify/fastify/pull/6879) |

package/fastify.d.ts

@@ -10,21 +10,20 @@ import { ConstraintStrategy, Config as FindMyWayConfig, HTTPVersion } from 'find
 import { InjectOptions, CallbackFunc as LightMyRequestCallback, Chain as LightMyRequestChain, Response as LightMyRequestResponse } from 'light-my-request'
 
 import { AddContentTypeParser, ConstructorAction, FastifyBodyParser, FastifyContentTypeParser, getDefaultJsonParser, hasContentTypeParser, ProtoAction } from './types/content-type-parser'
-import { FastifyContextConfig, FastifyReplyContext, FastifyRequestContext } from './types/context'
+import { FastifyContextConfig } from './types/context'
 import { FastifyErrorCodes } from './types/errors'
 import { DoneFuncWithErrOrRes, HookHandlerDoneFunction, onCloseAsyncHookHandler, onCloseHookHandler, onErrorAsyncHookHandler, onErrorHookHandler, onListenAsyncHookHandler, onListenHookHandler, onReadyAsyncHookHandler, onReadyHookHandler, onRegisterHookHandler, onRequestAbortAsyncHookHandler, onRequestAbortHookHandler, onRequestAsyncHookHandler, onRequestHookHandler, onResponseAsyncHookHandler, onResponseHookHandler, onRouteHookHandler, onSendAsyncHookHandler, onSendHookHandler, onTimeoutAsyncHookHandler, onTimeoutHookHandler, preCloseAsyncHookHandler, preCloseHookHandler, preHandlerAsyncHookHandler, preHandlerHookHandler, preParsingAsyncHookHandler, preParsingHookHandler, preSerializationAsyncHookHandler, preSerializationHookHandler, preValidationAsyncHookHandler, preValidationHookHandler, RequestPayload } from './types/hooks'
 import { FastifyInstance, FastifyListenOptions, PrintRoutesOptions } from './types/instance'
 import {
   FastifyBaseLogger,
   FastifyChildLoggerFactory,
-  LogController as LogControllerClass,
   FastifyLogFn,
-  FastifyLoggerInstance,
   FastifyLoggerOptions,
+  LogController as LogControllerClass,
   LogLevel,
   PinoLoggerOptions
 } from './types/logger'
-import { FastifyPlugin, FastifyPluginAsync, FastifyPluginCallback, FastifyPluginOptions } from './types/plugin'
+import { FastifyPluginAsync, FastifyPluginCallback, FastifyPluginOptions } from './types/plugin'
 import { FastifyRegister, FastifyRegisterOptions, RegisterOptions } from './types/register'
 import { FastifyReply } from './types/reply'
 import { FastifyRequest, RequestGenericInterface } from './types/request'
@@ -96,7 +95,10 @@ declare namespace fastify {
 
   type TrustProxyFunction = (address: string, hop: number) => boolean
 
-  export type FastifyRouterOptions<RawServer extends RawServerBase> = Omit<FindMyWayConfigForServer<RawServer>, 'defaultRoute' | 'onBadUrl' | 'onMaxParamLength' | 'querystringParser'> & {
+  export type FastifyRouterOptions<RawServer extends RawServerBase> = Omit<FindMyWayConfigForServer<RawServer>, 'defaultRoute' | 'onBadUrl' | 'onMaxParamLength' | 'querystringParser' | 'constraints'> & {
+    constraints?: {
+      [name: string]: ConstraintStrategy<FindMyWayVersion<RawServer>, unknown>,
+    },
     defaultRoute?: (
       req: RawRequestDefaultExpression<RawServer>,
       res: RawReplyDefaultExpression<RawServer>
@@ -132,8 +134,6 @@ declare namespace fastify {
     bodyLimit?: number,
     handlerTimeout?: number,
     maxParamLength?: number,
-    /** @deprecated Use the `logController` option with `disableRequestLogging` or `isLogDisabled` override instead. Will be removed in `fastify@6`. */
-    disableRequestLogging?: boolean | ((req: FastifyRequest) => boolean),
     logController?: LogControllerClass,
     exposeHeadRoutes?: boolean,
     onProtoPoisoning?: ProtoAction,
@@ -142,18 +142,9 @@ declare namespace fastify {
     loggerInstance?: Logger
     serializerOpts?: FJSOptions | Record<string, unknown>,
     serverFactory?: FastifyServerFactory<RawServer>,
-    caseSensitive?: boolean,
-    allowUnsafeRegex?: boolean,
     requestIdHeader?: string | false,
-    /** @deprecated Use the `logController` option with `requestIdLogLabel` instead. Will be removed in `fastify@6`. */
-    requestIdLogLabel?: string;
-    useSemicolonDelimiter?: boolean,
     genReqId?: (req: RawRequestDefaultExpression<RawServer>) => string,
     trustProxy?: boolean | string | string[] | number | TrustProxyFunction,
-    querystringParser?: (str: string) => { [key: string]: unknown },
-    constraints?: {
-      [name: string]: ConstraintStrategy<FindMyWayVersion<RawServer>, unknown>,
-    },
     schemaController?: {
       bucket?: (parentSchemas?: unknown) => {
         add(schema: unknown): FastifyInstance;
@@ -212,20 +203,15 @@ declare namespace fastify {
     routerOptions?: FastifyRouterOptions<RawServer>,
   }
 
-  /**
-   * @deprecated use {@link FastifySchemaValidationError}
-   */
-  export type ValidationResult = FastifySchemaValidationError
-
   /* Export additional types */
   export type {
     LightMyRequestChain, InjectOptions, LightMyRequestResponse, LightMyRequestCallback, // 'light-my-request'
     FastifyRequest, RequestGenericInterface, // './types/request'
     FastifyReply, // './types/reply'
-    FastifyPluginCallback, FastifyPluginAsync, FastifyPluginOptions, FastifyPlugin, // './types/plugin'
+    FastifyPluginCallback, FastifyPluginAsync, FastifyPluginOptions, // './types/plugin'
     FastifyListenOptions, FastifyInstance, PrintRoutesOptions, // './types/instance'
-    FastifyLoggerOptions, FastifyBaseLogger, FastifyLoggerInstance, FastifyLogFn, LogLevel, // './types/logger'
-    FastifyRequestContext, FastifyContextConfig, FastifyReplyContext, // './types/context'
+    FastifyLoggerOptions, FastifyBaseLogger, FastifyLogFn, LogLevel, // './types/logger'
+    FastifyContextConfig, // './types/context'
     RouteHandler, RouteHandlerMethod, RouteOptions, RouteShorthandMethod, RouteShorthandOptions,
     RouteShorthandOptionsWithHandler, RouteGenericInterface, // './types/route'
     FastifyRegister, FastifyRegisterOptions, RegisterOptions, // './types/register'

package/fastify.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const VERSION = '5.11.2'
+const VERSION = '6.0.0-alpha.0'
 
 const Avvio = require('avvio')
 const http = require('node:http')
@@ -57,7 +57,6 @@ const {
   AVVIO_ERRORS_MAP,
   ...errorCodes
 } = require('./lib/errors')
-const PonyPromise = require('./lib/promise')
 
 const { defaultInitOptions } = getSecuredInitialConfig
 
@@ -70,17 +69,17 @@ const {
   FST_ERR_SCHEMA_CONTROLLER_BUCKET_OPT_NOT_FN,
   FST_ERR_AJV_CUSTOM_OPTIONS_OPT_NOT_OBJ,
   FST_ERR_AJV_CUSTOM_OPTIONS_OPT_NOT_ARR,
-  FST_ERR_INSTANCE_ALREADY_LISTENING,
+  FST_ERR_INSTANCE_ALREADY_STARTED,
   FST_ERR_REOPENED_CLOSE_SERVER,
   FST_ERR_ROUTE_REWRITE_NOT_STR,
   FST_ERR_SCHEMA_ERROR_FORMATTER_NOT_FN,
   FST_ERR_ERROR_HANDLER_NOT_FN,
   FST_ERR_ERROR_HANDLER_ALREADY_SET,
-  FST_ERR_ROUTE_METHOD_INVALID
+  FST_ERR_ROUTE_METHOD_INVALID,
+  FST_ERR_ROUTE_METHOD_ALREADY_SUPPORTED
 } = errorCodes
 
 const { buildErrorHandler } = require('./lib/error-handler.js')
-const { FSTWRN004, FSTDEP023, FSTDEP024, FSTDEP025 } = require('./lib/warnings.js')
 
 const initChannel = diagnostics.channel('fastify.initialization')
 
@@ -466,7 +465,7 @@ function fastify (serverOptions) {
   return fastify
 
   function throwIfAlreadyStarted (msg) {
-    if (fastify[kState].started) throw new FST_ERR_INSTANCE_ALREADY_LISTENING(msg)
+    if (fastify[kState].started) throw new FST_ERR_INSTANCE_ALREADY_STARTED(msg)
   }
 
   // HTTP injection handling
@@ -528,7 +527,7 @@ function fastify (serverOptions) {
     // It will work as a barrier for all the .ready() calls (ensuring single hook execution)
     // as well as a flow control mechanism to chain cbs and further
     // promises
-    this[kState].readyResolver = PonyPromise.withResolvers()
+    this[kState].readyResolver = Promise.withResolvers()
 
     if (!cb) {
       return this[kState].readyResolver.promise
@@ -771,10 +770,8 @@ function fastify (serverOptions) {
       throw new FST_ERR_ERROR_HANDLER_NOT_FN()
     }
 
-    if (!options.allowErrorHandlerOverride && this[kErrorHandlerAlreadySet]) {
+    if (this[kErrorHandlerAlreadySet] && !options.allowErrorHandlerOverride) {
       throw new FST_ERR_ERROR_HANDLER_ALREADY_SET()
-    } else if (this[kErrorHandlerAlreadySet]) {
-      FSTWRN004()
     }
 
     this[kErrorHandlerAlreadySet] = true
@@ -832,7 +829,7 @@ function fastify (serverOptions) {
       this[kSupportedHTTPMethods].bodywith.has(method)
 
     if (alreadyExists && !overrideExisting) {
-      FSTDEP025(method)
+      throw new FST_ERR_ROUTE_METHOD_ALREADY_SUPPORTED(method)
     }
 
     if (hasBody === true) {
@@ -864,13 +861,10 @@ function processOptions (options, defaultRoute, onBadUrl, onMaxParamLength) {
   }
 
   if (
-    (options.querystringParser && typeof options.querystringParser !== 'function') ||
-    (
-      options.routerOptions?.querystringParser &&
+    options.routerOptions?.querystringParser &&
       typeof options.routerOptions.querystringParser !== 'function'
-    )
   ) {
-    throw new FST_ERR_QSP_NOT_FN(typeof (options.querystringParser ?? options.routerOptions.querystringParser))
+    throw new FST_ERR_QSP_NOT_FN(typeof options.routerOptions.querystringParser)
   }
 
   if (options.schemaController && options.schemaController.bucket && typeof options.schemaController.bucket !== 'function') {
@@ -881,13 +875,7 @@ function processOptions (options, defaultRoute, onBadUrl, onMaxParamLength) {
 
   const requestIdHeader = typeof options.requestIdHeader === 'string' && options.requestIdHeader.length !== 0 ? options.requestIdHeader.toLowerCase() : (options.requestIdHeader === true && 'request-id')
   const genReqId = reqIdGenFactory(requestIdHeader, options.genReqId)
-  if (options.requestIdLogLabel !== undefined) {
-    FSTDEP024()
-  }
   options.bodyLimit = options.bodyLimit || defaultInitOptions.bodyLimit
-  if (options.disableRequestLogging !== undefined) {
-    FSTDEP023()
-  }
 
   const ajvOptions = Object.assign({
     customOptions: {},
@@ -918,6 +906,13 @@ function processOptions (options, defaultRoute, onBadUrl, onMaxParamLength) {
   options.clientErrorHandler = options.clientErrorHandler || defaultClientErrorHandler
   options.allowErrorHandlerOverride = options.allowErrorHandlerOverride ?? defaultInitOptions.allowErrorHandlerOverride
 
+  options.routerOptions = buildRouterOptions(options, {
+    buildPrettyMeta: defaultBuildPrettyMeta,
+    defaultRoute,
+    onBadUrl,
+    onMaxParamLength
+  })
+
   const initialConfig = getSecuredInitialConfig(options)
 
   // exposeHeadRoutes have its default set from the validator
@@ -926,18 +921,6 @@ function processOptions (options, defaultRoute, onBadUrl, onMaxParamLength) {
   // we need to set this before calling createServer
   options.http2SessionTimeout = initialConfig.http2SessionTimeout
 
-  options.routerOptions = buildRouterOptions(options, {
-    defaultRoute,
-    onBadUrl,
-    onMaxParamLength,
-    ignoreTrailingSlash: defaultInitOptions.ignoreTrailingSlash,
-    ignoreDuplicateSlashes: defaultInitOptions.ignoreDuplicateSlashes,
-    maxParamLength: defaultInitOptions.maxParamLength,
-    allowUnsafeRegex: defaultInitOptions.allowUnsafeRegex,
-    buildPrettyMeta: defaultBuildPrettyMeta,
-    useSemicolonDelimiter: defaultInitOptions.useSemicolonDelimiter
-  })
-
   return {
     options,
     genReqId,