fastify 5.12.0 → 6.0.0-alpha.1

package/fastify.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const VERSION = '5.12.0'
+const VERSION = '6.0.0-alpha.1'
 
 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,