fastify 5.12.0 → 6.0.0-alpha.1

package/lib/errors.js

@@ -416,6 +416,12 @@ const codes = {
     500,
     TypeError
   ),
+  FST_ERR_ROUTE_METHOD_ALREADY_SUPPORTED: createError(
+    'FST_ERR_ROUTE_METHOD_ALREADY_SUPPORTED',
+    'Method "%s" is already supported. Use `overrideExisting: true` to override it.',
+    500,
+    TypeError
+  ),
   FST_ERR_ROUTE_METHOD_NOT_SUPPORTED: createError(
     'FST_ERR_ROUTE_METHOD_NOT_SUPPORTED',
     '%s method is not supported.',
@@ -475,10 +481,10 @@ const codes = {
   ),
   FST_ERR_REOPENED_SERVER: createError(
     'FST_ERR_REOPENED_SERVER',
-    'Fastify is already listening'
+    'Fastify is already started'
   ),
-  FST_ERR_INSTANCE_ALREADY_LISTENING: createError(
-    'FST_ERR_INSTANCE_ALREADY_LISTENING',
+  FST_ERR_INSTANCE_ALREADY_STARTED: createError(
+    'FST_ERR_INSTANCE_ALREADY_STARTED',
     'Fastify instance is already listening. %s'
   ),
 

package/lib/log-controller.js

@@ -1,7 +1,5 @@
 'use strict'
 
-const { defaultInitOptions } = require('./initial-config-validation')
-
 /**
  * Internal logger layer that wraps Fastify's log calls with
  * request-lifecycle awareness (disable-check, level selection).
@@ -20,9 +18,9 @@ class LogController {
    */
   constructor (options) {
     const opts = options || {}
-    this.disableRequestLogging = opts.disableRequestLogging || defaultInitOptions.disableRequestLogging
+    this.disableRequestLogging = opts.disableRequestLogging || false
     this.isDisableRequestLoggingFunction = typeof this.disableRequestLogging === 'function'
-    this.requestIdLogLabel = opts.requestIdLogLabel || defaultInitOptions.requestIdLogLabel
+    this.requestIdLogLabel = opts.requestIdLogLabel || 'reqId'
   }
 
   /**

package/lib/logger-factory.js

@@ -132,7 +132,7 @@ function createLogger (options) {
 function createLogController (options) {
   const userController = options.logController
   if (!userController) {
-    return new LogController(options)
+    return new LogController()
   }
 
   if (userController instanceof LogController) {

package/lib/route.js

@@ -6,6 +6,7 @@ const handleRequest = require('./handle-request.js')
 const { onRequestAbortHookRunner, lifecycleHooks, preParsingHookRunner, onTimeoutHookRunner, onRequestHookRunner } = require('./hooks')
 const { normalizeSchema } = require('./schemas')
 const { parseHeadOnSendHandlers } = require('./head-route.js')
+const { defaultInitOptions } = require('./initial-config-validation.js')
 
 const {
   compileSchemasForValidation,
@@ -57,8 +58,6 @@ const {
 const { buildErrorHandler } = require('./error-handler')
 const { createChildLogger, defaultChildLoggerFactory } = require('./logger-factory.js')
 const { getGenReqId } = require('./req-id-gen-factory.js')
-const { FSTDEP022 } = require('./warnings')
-
 const routerKeys = [
   'allowUnsafeRegex',
   'buildPrettyMeta',
@@ -663,15 +662,11 @@ function buildRouterOptions (options, defaultOptions) {
     ? Object.create(null)
     : Object.assign(Object.create(null), options.routerOptions)
 
-  const usedDeprecatedOptions = routerKeys.filter(key => Object.hasOwn(options, key))
-
-  if (usedDeprecatedOptions.length > 0) {
-    FSTDEP022(usedDeprecatedOptions.join(', '))
-  }
-
   for (const key of routerKeys) {
     if (!Object.hasOwn(routerOptions, key)) {
-      routerOptions[key] = options[key] ?? defaultOptions[key]
+      // defaultInitOptions are non-function default
+      // defaultOptions are function default
+      routerOptions[key] = options[key] ?? defaultInitOptions.routerOptions[key] ?? defaultOptions[key]
     }
   }
 

package/lib/server.js

@@ -16,7 +16,6 @@ const {
   FST_ERR_FORCE_CLOSE_CONNECTIONS_IDLE_NOT_AVAILABLE
 } = require('./errors')
 const noopSet = require('./noop-set')
-const PonyPromise = require('./promise')
 
 module.exports.createServer = createServer
 
@@ -106,7 +105,7 @@ function createServer (options, httpHandler) {
     if (cb === undefined) {
       const listening = listenPromise.call(this, server, listenOptions)
       return listening.then(address => {
-        const { promise, resolve } = PonyPromise.withResolvers()
+        const { promise, resolve } = Promise.withResolvers()
         if (host === 'localhost') {
           multipleBindings.call(this, server, httpHandler, options, listenOptions, () => {
             this[kState].listening = true
@@ -277,7 +276,7 @@ function listenPromise (server, listenOptions) {
     // skip listen when aborted during ready
     if (this[kState].aborted) return
 
-    const { promise, resolve, reject } = PonyPromise.withResolvers()
+    const { promise, resolve, reject } = Promise.withResolvers()
 
     const errEventHandler = (err) => {
       cleanup()

package/lib/warnings.js

@@ -3,13 +3,9 @@
 const { createWarning } = require('process-warning')
 
 /**
- * Deprecation codes:
+ * Warning codes:
  *   - FSTWRN001
  *   - FSTSEC001
- *   - FSTDEP022
- *   - FSTDEP023
- *   - FSTDEP024
- *   - FSTDEP025
  *
  * Deprecation Codes FSTDEP001 - FSTDEP021 were used by v4 and MUST NOT be reused.
  *                             - FSTDEP022 - FSTDEP025 are used by v5 and MUST NOT be reused.
@@ -30,13 +26,6 @@ const FSTWRN003 = createWarning({
   unlimited: true
 })
 
-const FSTWRN004 = createWarning({
-  name: 'FastifyWarning',
-  code: 'FSTWRN004',
-  message: 'It seems that you are overriding an errorHandler in the same scope, which can lead to subtle bugs. To disable this behavior, set \'allowErrorHandlerOverride\' to false. For more information, visit: https://fastify.dev/docs/latest/Reference/Server/#allowerrorhandleroverride',
-  unlimited: true
-})
-
 const FSTSEC001 = createWarning({
   name: 'FastifySecurity',
   code: 'FSTSEC001',
@@ -44,41 +33,8 @@ const FSTSEC001 = createWarning({
   unlimited: true
 })
 
-const FSTDEP022 = createWarning({
-  name: 'FastifyWarning',
-  code: 'FSTDEP022',
-  message: 'The router options for %s property access is deprecated. Please use "options.routerOptions" instead for accessing router options. The router options will be removed in `fastify@6`.',
-  unlimited: true
-})
-
-const FSTDEP023 = createWarning({
-  name: 'FastifyDeprecation',
-  code: 'FSTDEP023',
-  message: 'disableRequestLogging option is deprecated. Use the logController option with disableRequestLogging or isLogDisabled override instead. The disableRequestLogging top-level option will be removed in `fastify@6`.',
-  unlimited: true
-})
-
-const FSTDEP024 = createWarning({
-  name: 'FastifyDeprecation',
-  code: 'FSTDEP024',
-  message: 'requestIdLogLabel option is deprecated. Use the logController option with requestIdLogLabel instead. The requestIdLogLabel top-level option will be removed in `fastify@6`.',
-  unlimited: true
-})
-
-const FSTDEP025 = createWarning({
-  name: 'FastifyDeprecation',
-  code: 'FSTDEP025',
-  message: 'Calling addHttpMethod for existing method "%s" without { overrideExisting: true } is deprecated. In Fastify v6, this will throw an error.',
-  unlimited: true
-})
-
 module.exports = {
   FSTWRN001,
   FSTWRN003,
-  FSTWRN004,
-  FSTSEC001,
-  FSTDEP022,
-  FSTDEP023,
-  FSTDEP024,
-  FSTDEP025
+  FSTSEC001
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fastify",
-  "version": "5.12.0",
+  "version": "6.0.0-alpha.1",
   "description": "Fast and low overhead web framework, for Node.js",
   "main": "fastify.js",
   "type": "commonjs",
@@ -198,7 +198,7 @@
     "tstyche": "^7.0.0",
     "typebox": "^1.0.81",
     "typescript": "~6.0.2",
-    "undici": "^7.11.0",
+    "undici": "^8.0.0",
     "vary": "^1.1.2",
     "yup": "^1.4.0"
   },

package/test/404s.test.js

@@ -1984,52 +1984,3 @@ test('hooks are applied to not found handlers /3', async t => {
   const { statusCode } = await fastify.inject('/')
   t.assert.strictEqual(statusCode, 401)
 })
-
-test('should honor disableRequestLogging function for 404', async t => {
-  t.plan(3)
-
-  const Writable = require('node:stream').Writable
-
-  const logStream = new Writable()
-  logStream.logs = []
-  logStream._write = function (chunk, encoding, callback) {
-    this.logs.push(JSON.parse(chunk.toString()))
-    callback()
-  }
-
-  const fastify = Fastify({
-    logger: {
-      level: 'info',
-      stream: logStream
-    },
-    disableRequestLogging: (req) => {
-      // Disable logging for URLs containing 'silent'
-      return req.url.includes('silent')
-    }
-  })
-
-  fastify.get('/', function (req, reply) {
-    reply.send({ hello: 'world' })
-  })
-
-  t.after(() => { fastify.close() })
-
-  // First request to a non-existent route (no 'silent' in URL) - should log
-  const response1 = await fastify.inject({
-    method: 'GET',
-    url: '/not-found'
-  })
-  t.assert.strictEqual(response1.statusCode, 404)
-
-  // Second request to a non-existent route with 'silent' in URL - should not log
-  const response2 = await fastify.inject({
-    method: 'GET',
-    url: '/silent-route'
-  })
-  t.assert.strictEqual(response2.statusCode, 404)
-
-  // Check logs: first request should have logged, second should not
-  // We expect: incoming request, Route not found info, request completed (for first request only)
-  const infoLogs = logStream.logs.filter(log => log.msg && log.msg.includes('Route GET:/not-found not found'))
-  t.assert.strictEqual(infoLogs.length, 1, 'Should log 404 info only for non-silent route')
-})

package/test/allow-unsafe-regex.test.js

@@ -7,7 +7,9 @@ test('allow unsafe regex', async t => {
   t.plan(2)
 
   const fastify = Fastify({
-    allowUnsafeRegex: false
+    routerOptions: {
+      allowUnsafeRegex: false
+    }
   })
   t.after(() => fastify.close())
 
@@ -26,7 +28,9 @@ test('allow unsafe regex not match', async t => {
   t.plan(1)
 
   const fastify = Fastify({
-    allowUnsafeRegex: false
+    routerOptions: {
+      allowUnsafeRegex: false
+    }
   })
   t.after(() => fastify.close())
 
@@ -44,7 +48,9 @@ test('allow unsafe regex not safe', (t, done) => {
   t.plan(1)
 
   const fastify = Fastify({
-    allowUnsafeRegex: false
+    routerOptions: {
+      allowUnsafeRegex: false
+    }
   })
   t.after(() => fastify.close())
 
@@ -74,7 +80,9 @@ test('allow unsafe regex allow unsafe', async t => {
   t.plan(3)
 
   const fastify = Fastify({
-    allowUnsafeRegex: true
+    routerOptions: {
+      allowUnsafeRegex: true
+    }
   })
   t.after(() => fastify.close())
 

package/test/case-insensitive.test.js

@@ -7,7 +7,9 @@ test('case insensitive', async (t) => {
   t.plan(3)
 
   const fastify = Fastify({
-    caseSensitive: false
+    routerOptions: {
+      caseSensitive: false
+    }
   })
   t.after(() => fastify.close())
 
@@ -30,7 +32,9 @@ test('case insensitive inject', async t => {
   t.plan(2)
 
   const fastify = Fastify({
-    caseSensitive: false
+    routerOptions: {
+      caseSensitive: false
+    }
   })
   t.after(() => fastify.close())
 
@@ -55,7 +59,9 @@ test('case insensitive (parametric)', async (t) => {
   t.plan(4)
 
   const fastify = Fastify({
-    caseSensitive: false
+    routerOptions: {
+      caseSensitive: false
+    }
   })
   t.after(() => fastify.close())
 
@@ -81,7 +87,9 @@ test('case insensitive (wildcard)', async (t) => {
   t.plan(4)
 
   const fastify = Fastify({
-    caseSensitive: false
+    routerOptions: {
+      caseSensitive: false
+    }
   })
   t.after(() => fastify.close())
 

package/test/custom-querystring-parser.test.js

@@ -94,22 +94,6 @@ test('Querystring without value', async t => {
   t.assert.strictEqual(injectResponse.statusCode, 200)
 })
 
-test('Custom querystring parser should be a function', t => {
-  t.plan(1)
-
-  try {
-    Fastify({
-      querystringParser: 10
-    })
-    t.assert.fail('Should throw')
-  } catch (err) {
-    t.assert.strictEqual(
-      err.message,
-      "querystringParser option should be a function, instead got 'number'"
-    )
-  }
-})
-
 test('Custom querystring parser should be a function', t => {
   t.plan(1)
 

package/test/hooks.test.js

@@ -674,7 +674,12 @@ test('onRoute hook should be called once when prefixTrailingSlash', (t, testDone
   let onRouteCalled = 0
   let routePatched = 0
 
-  const fastify = Fastify({ ignoreTrailingSlash: false, exposeHeadRoutes: false })
+  const fastify = Fastify({
+    exposeHeadRoutes: false,
+    routerOptions: {
+      ignoreTrailingSlash: false
+    }
+  })
 
   // a plugin that patches route options, similar to fastify-compress
   fastify.register(fp(function myPlugin (instance, opts, next) {

package/test/http-methods/custom-http-methods.test.js

@@ -113,36 +113,31 @@ test('addHttpMethod rejects fake http method', t => {
   t.assert.throws(() => { fastify.addHttpMethod('FOOO') }, /Provided method is invalid!/)
 })
 
-test('addHttpMethod warns when overriding an existing method', (t, done) => {
+test('addHttpMethod rejects an implicit override of an existing method', t => {
   const fastify = Fastify()
-  const onWarning = warning => {
-    t.assert.strictEqual(warning.name, 'FastifyDeprecation')
-    t.assert.strictEqual(warning.code, 'FSTDEP025')
-    done()
-  }
-  process.once('warning', onWarning)
 
   t.after(() => {
     fastify.close()
-    process.removeListener('warning', onWarning)
   })
 
-  fastify.addHttpMethod('GET', { hasBody: true })
+  t.assert.throws(
+    () => fastify.addHttpMethod('GET', { hasBody: true }),
+    {
+      code: 'FST_ERR_ROUTE_METHOD_ALREADY_SUPPORTED',
+      message: 'Method "GET" is already supported. Use `overrideExisting: true` to override it.'
+    }
+  )
 })
 
-test('addHttpMethod does not warn when overriding an existing method explicitly', t => {
-  const doNotWarn = () => {
-    t.assert.fail('should not warn')
-  }
-  process.on('warning', doNotWarn)
-
+test('addHttpMethod allows an explicit override of an existing method', t => {
   const fastify = Fastify()
   t.after(() => {
     fastify.close()
-    process.removeListener('warning', doNotWarn)
   })
 
-  fastify.addHttpMethod('POST', { overrideExisting: true })
+  t.assert.doesNotThrow(() => {
+    fastify.addHttpMethod('POST', { overrideExisting: true })
+  })
 })
 
 test('addHttpMethod can change an existing method body behavior', async t => {

package/test/http-methods/trace.test.js

@@ -2,7 +2,7 @@
 
 const { test } = require('node:test')
 const fastify = require('../../fastify')()
-fastify.addHttpMethod('TRACE')
+fastify.addHttpMethod('TRACE', { overrideExisting: true })
 
 test('shorthand - trace', t => {
   t.plan(1)

package/test/https/custom-https-server.test.js

@@ -5,7 +5,7 @@ const Fastify = require('../..')
 const https = require('node:https')
 const dns = require('node:dns').promises
 const { buildCertificate } = require('../build-certificate')
-const { Agent } = require('undici')
+const { Agent, fetch } = require('undici')
 
 async function setup () {
   await buildCertificate()

package/test/internals/errors.test.js

@@ -5,7 +5,7 @@ const errors = require('../../lib/errors')
 const { readFileSync } = require('node:fs')
 const { resolve } = require('node:path')
 
-const expectedErrors = 94
+const expectedErrors = 95
 
 test(`should expose ${expectedErrors} errors`, t => {
   t.plan(1)
@@ -710,6 +710,16 @@ test('FST_ERR_ROUTE_METHOD_INVALID', t => {
   t.assert.ok(error instanceof TypeError)
 })
 
+test('FST_ERR_ROUTE_METHOD_ALREADY_SUPPORTED', t => {
+  t.plan(5)
+  const error = new errors.FST_ERR_ROUTE_METHOD_ALREADY_SUPPORTED('GET')
+  t.assert.strictEqual(error.name, 'FastifyError')
+  t.assert.strictEqual(error.code, 'FST_ERR_ROUTE_METHOD_ALREADY_SUPPORTED')
+  t.assert.strictEqual(error.message, 'Method "GET" is already supported. Use `overrideExisting: true` to override it.')
+  t.assert.strictEqual(error.statusCode, 500)
+  t.assert.ok(error instanceof TypeError)
+})
+
 test('FST_ERR_ROUTE_METHOD_NOT_SUPPORTED', t => {
   t.plan(5)
   const error = new errors.FST_ERR_ROUTE_METHOD_NOT_SUPPORTED()
@@ -805,16 +815,16 @@ test('FST_ERR_REOPENED_SERVER', t => {
   const error = new errors.FST_ERR_REOPENED_SERVER()
   t.assert.strictEqual(error.name, 'FastifyError')
   t.assert.strictEqual(error.code, 'FST_ERR_REOPENED_SERVER')
-  t.assert.strictEqual(error.message, 'Fastify is already listening')
+  t.assert.strictEqual(error.message, 'Fastify is already started')
   t.assert.strictEqual(error.statusCode, 500)
   t.assert.ok(error instanceof Error)
 })
 
-test('FST_ERR_INSTANCE_ALREADY_LISTENING', t => {
+test('FST_ERR_INSTANCE_ALREADY_STARTED', t => {
   t.plan(5)
-  const error = new errors.FST_ERR_INSTANCE_ALREADY_LISTENING()
+  const error = new errors.FST_ERR_INSTANCE_ALREADY_STARTED()
   t.assert.strictEqual(error.name, 'FastifyError')
-  t.assert.strictEqual(error.code, 'FST_ERR_INSTANCE_ALREADY_LISTENING')
+  t.assert.strictEqual(error.code, 'FST_ERR_INSTANCE_ALREADY_STARTED')
   t.assert.strictEqual(error.message, 'Fastify instance is already listening. %s')
   t.assert.strictEqual(error.statusCode, 500)
   t.assert.ok(error instanceof Error)

package/test/internals/initial-config.test.js

@@ -36,20 +36,21 @@ test('without options passed to Fastify, initialConfig should expose default val
     requestTimeout: 0,
     handlerTimeout: 0,
     bodyLimit: 1024 * 1024,
-    caseSensitive: true,
-    allowUnsafeRegex: false,
-    disableRequestLogging: false,
-    ignoreTrailingSlash: false,
-    ignoreDuplicateSlashes: false,
-    maxParamLength: 100,
     onProtoPoisoning: 'error',
     onConstructorPoisoning: 'error',
     pluginTimeout: 10000,
     requestIdHeader: false,
-    requestIdLogLabel: 'reqId',
     http2SessionTimeout: 72000,
     exposeHeadRoutes: true,
-    useSemicolonDelimiter: false
+    routerOptions: {
+      allowUnsafeRegex: false,
+      caseSensitive: true,
+      constraints: undefined,
+      ignoreTrailingSlash: false,
+      ignoreDuplicateSlashes: false,
+      maxParamLength: 100,
+      useSemicolonDelimiter: false
+    }
   }
 
   t.assert.deepStrictEqual(Fastify().initialConfig, fastifyDefaultOptions)
@@ -88,65 +89,79 @@ test('Fastify.initialConfig should expose all options', t => {
       key: global.context.key,
       cert: global.context.cert
     },
-    ignoreTrailingSlash: true,
-    ignoreDuplicateSlashes: true,
-    maxParamLength: 200,
     connectionTimeout: 0,
     keepAliveTimeout: 72000,
     bodyLimit: 1049600,
     onProtoPoisoning: 'remove',
     serverFactory,
-    caseSensitive: true,
-    allowUnsafeRegex: false,
     requestIdHeader: 'request-id-alt',
     pluginTimeout: 20000,
-    useSemicolonDelimiter: false,
-    querystringParser: str => str,
     genReqId: function (req) {
       return reqId++
     },
     loggerInstance: pino({ level: 'info' }),
-    constraints: {
-      version: versionStrategy
-    },
     trustProxy: function myTrustFn (address, hop) {
       return address === '1.2.3.4' || hop === 1
+    },
+    routerOptions: {
+      allowUnsafeRegex: false,
+      caseSensitive: true,
+      constraints: {
+        version: versionStrategy
+      },
+      ignoreTrailingSlash: true,
+      ignoreDuplicateSlashes: true,
+      maxParamLength: 200,
+      querystringParser: str => str,
+      useSemicolonDelimiter: false
     }
   }
 
   const fastify = Fastify(options)
+
   t.assert.strictEqual(fastify.initialConfig.http2, true)
   t.assert.strictEqual(fastify.initialConfig.https, true, 'for security reason the key cert is hidden')
-  t.assert.strictEqual(fastify.initialConfig.ignoreTrailingSlash, true)
-  t.assert.strictEqual(fastify.initialConfig.ignoreDuplicateSlashes, true)
-  t.assert.strictEqual(fastify.initialConfig.maxParamLength, 200)
   t.assert.strictEqual(fastify.initialConfig.connectionTimeout, 0)
   t.assert.strictEqual(fastify.initialConfig.keepAliveTimeout, 72000)
   t.assert.strictEqual(fastify.initialConfig.bodyLimit, 1049600)
   t.assert.strictEqual(fastify.initialConfig.onProtoPoisoning, 'remove')
-  t.assert.strictEqual(fastify.initialConfig.caseSensitive, true)
-  t.assert.strictEqual(fastify.initialConfig.useSemicolonDelimiter, false)
-  t.assert.strictEqual(fastify.initialConfig.allowUnsafeRegex, false)
   t.assert.strictEqual(fastify.initialConfig.requestIdHeader, 'request-id-alt')
   t.assert.strictEqual(fastify.initialConfig.pluginTimeout, 20000)
-  t.assert.ok(fastify.initialConfig.constraints.version)
 
   // obfuscated options:
   t.assert.strictEqual(fastify.initialConfig.serverFactory, undefined)
   t.assert.strictEqual(fastify.initialConfig.trustProxy, undefined)
   t.assert.strictEqual(fastify.initialConfig.genReqId, undefined)
   t.assert.strictEqual(fastify.initialConfig.childLoggerFactory, undefined)
-  t.assert.strictEqual(fastify.initialConfig.querystringParser, undefined)
   t.assert.strictEqual(fastify.initialConfig.logger, undefined)
   t.assert.strictEqual(fastify.initialConfig.trustProxy, undefined)
+
+  // router options
+  t.assert.strictEqual(fastify.initialConfig.routerOptions.allowUnsafeRegex, false)
+  t.assert.strictEqual(fastify.initialConfig.routerOptions.caseSensitive, true)
+  t.assert.ok(fastify.initialConfig.routerOptions.constraints.version)
+  t.assert.strictEqual(fastify.initialConfig.routerOptions.ignoreTrailingSlash, true)
+  t.assert.strictEqual(fastify.initialConfig.routerOptions.ignoreDuplicateSlashes, true)
+  t.assert.strictEqual(fastify.initialConfig.routerOptions.maxParamLength, 200)
+  t.assert.strictEqual(fastify.initialConfig.routerOptions.querystringParser, undefined)
+  t.assert.strictEqual(fastify.initialConfig.routerOptions.useSemicolonDelimiter, false)
 })
 
 test('Should throw if you try to modify Fastify.initialConfig', t => {
-  t.plan(4)
+  t.plan(8)
 
-  const fastify = Fastify({ ignoreTrailingSlash: true })
+  const fastify = Fastify({ keepAliveTimeout: 1, routerOptions: { ignoreTrailingSlash: true } })
   try {
-    fastify.initialConfig.ignoreTrailingSlash = false
+    fastify.initialConfig.keepAliveTimeout = 2
+    t.assert.fail()
+  } catch (error) {
+    t.assert.ok(error instanceof TypeError)
+    t.assert.strictEqual(error.message, "Cannot assign to read only property 'keepAliveTimeout' of object '#<Object>'")
+    t.assert.ok(error.stack)
+    t.assert.ok(true)
+  }
+  try {
+    fastify.initialConfig.routerOptions.ignoreTrailingSlash = false
     t.assert.fail()
   } catch (error) {
     t.assert.ok(error instanceof TypeError)
@@ -252,10 +267,12 @@ test('Should not have issues when passing stream options to Pino.js', (t, done)
   const stream = split(JSON.parse)
 
   const originalOptions = {
-    ignoreTrailingSlash: true,
     logger: {
       level: 'trace',
       stream
+    },
+    routerOptions: {
+      ignoreTrailingSlash: true
     }
   }
 
@@ -276,20 +293,21 @@ test('Should not have issues when passing stream options to Pino.js', (t, done)
       requestTimeout: 0,
       handlerTimeout: 0,
       bodyLimit: 1024 * 1024,
-      caseSensitive: true,
-      allowUnsafeRegex: false,
-      disableRequestLogging: false,
-      ignoreTrailingSlash: true,
-      ignoreDuplicateSlashes: false,
-      maxParamLength: 100,
       onProtoPoisoning: 'error',
       onConstructorPoisoning: 'error',
       pluginTimeout: 10000,
       requestIdHeader: false,
-      requestIdLogLabel: 'reqId',
       http2SessionTimeout: 72000,
       exposeHeadRoutes: true,
-      useSemicolonDelimiter: false
+      routerOptions: {
+        allowUnsafeRegex: false,
+        caseSensitive: true,
+        constraints: undefined,
+        ignoreTrailingSlash: true,
+        ignoreDuplicateSlashes: false,
+        maxParamLength: 100,
+        useSemicolonDelimiter: false
+      }
     })
   } catch (error) {
     t.assert.fail()