fastify 4.0.0-rc.3 → 4.0.0

package/fastify.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const VERSION = '4.0.0-rc.3'
+const VERSION = '4.0.0'
 
 const Avvio = require('avvio')
 const http = require('http')
@@ -52,6 +52,7 @@ const { defaultInitOptions } = getSecuredInitialConfig
 
 const {
   FST_ERR_BAD_URL,
+  FST_ERR_FORCE_CLOSE_CONNECTIONS_IDLE_NOT_AVAILABLE,
   AVVIO_ERRORS_MAP,
   appendStackTrace
 } = require('./lib/errors')
@@ -123,7 +124,6 @@ function fastify (options) {
   // Update the options with the fixed values
   options.connectionTimeout = options.connectionTimeout || defaultInitOptions.connectionTimeout
   options.keepAliveTimeout = options.keepAliveTimeout || defaultInitOptions.keepAliveTimeout
-  options.forceCloseConnections = typeof options.forceCloseConnections === 'boolean' ? options.forceCloseConnections : defaultInitOptions.forceCloseConnections
   options.maxRequestsPerSocket = options.maxRequestsPerSocket || defaultInitOptions.maxRequestsPerSocket
   options.requestTimeout = options.requestTimeout || defaultInitOptions.requestTimeout
   options.logger = logger
@@ -135,7 +135,6 @@ function fastify (options) {
   options.clientErrorHandler = options.clientErrorHandler || defaultClientErrorHandler
 
   const initialConfig = getSecuredInitialConfig(options)
-  const keepAliveConnections = options.forceCloseConnections === true ? new Set() : noopSet()
 
   // exposeHeadRoutes have its default set from the validator
   options.exposeHeadRoutes = initialConfig.exposeHeadRoutes
@@ -166,13 +165,13 @@ function fastify (options) {
       onBadUrl,
       constraints,
       ignoreTrailingSlash: options.ignoreTrailingSlash || defaultInitOptions.ignoreTrailingSlash,
+      ignoreDuplicateSlashes: options.ignoreDuplicateSlashes || defaultInitOptions.ignoreDuplicateSlashes,
       maxParamLength: options.maxParamLength || defaultInitOptions.maxParamLength,
       caseSensitive: options.caseSensitive,
       allowUnsafeRegex: options.allowUnsafeRegex || defaultInitOptions.allowUnsafeRegex,
       buildPrettyMeta: defaultBuildPrettyMeta,
       querystringParser: options.querystringParser
-    },
-    keepAliveConnections
+    }
   })
 
   // 404 router, used for handling encapsulated 404 handlers
@@ -185,6 +184,19 @@ function fastify (options) {
   options.http2SessionTimeout = initialConfig.http2SessionTimeout
   const { server, listen } = createServer(options, httpHandler)
 
+  const serverHasCloseAllConnections = typeof server.closeAllConnections === 'function'
+  const serverHasCloseIdleConnections = typeof server.closeIdleConnections === 'function'
+
+  let forceCloseConnections = options.forceCloseConnections
+  if (forceCloseConnections === 'idle' && !serverHasCloseIdleConnections) {
+    throw new FST_ERR_FORCE_CLOSE_CONNECTIONS_IDLE_NOT_AVAILABLE()
+  } else if (typeof forceCloseConnections !== 'boolean') {
+    /* istanbul ignore next: only one branch can be valid in a given Node.js version */
+    forceCloseConnections = serverHasCloseIdleConnections ? 'idle' : false
+  }
+
+  const keepAliveConnections = !serverHasCloseAllConnections && forceCloseConnections === true ? new Set() : noopSet()
+
   const setupResponseListeners = Reply.setupResponseListeners
   const schemaController = SchemaController.buildSchemaController(null, options.schemaController)
 
@@ -218,7 +230,7 @@ function fastify (options) {
     [kRequest]: Request.buildRequest(Request, options.trustProxy),
     [kFourOhFour]: fourOhFour,
     [pluginUtils.registeredPlugins]: [],
-    [kPluginNameChain]: [],
+    [kPluginNameChain]: ['fastify'],
     [kAvvioBoot]: null,
     // routing method
     routing: httpHandler,
@@ -284,6 +296,9 @@ function fastify (options) {
     onClose: null,
     close: null,
     printPlugins: null,
+    hasPlugin: function (name) {
+      return this[kPluginNameChain].includes(name)
+    },
     // http server
     listen,
     server,
@@ -385,13 +400,21 @@ function fastify (options) {
         // No new TCP connections are accepted
         instance.server.close(done)
 
-        for (const conn of fastify[kKeepAliveConnections]) {
-          // We must invoke the destroy method instead of merely unreffing
-          // the sockets. If we only unref, then the callback passed to
-          // `fastify.close` will never be invoked; nor will any of the
-          // registered `onClose` hooks.
-          conn.destroy()
-          fastify[kKeepAliveConnections].delete(conn)
+        /* istanbul ignore next: Cannot test this without Node.js core support */
+        if (forceCloseConnections === 'idle') {
+          instance.server.closeIdleConnections()
+        /* istanbul ignore next: Cannot test this without Node.js core support */
+        } else if (serverHasCloseAllConnections && forceCloseConnections) {
+          instance.server.closeAllConnections()
+        } else {
+          for (const conn of fastify[kKeepAliveConnections]) {
+            // We must invoke the destroy method instead of merely unreffing
+            // the sockets. If we only unref, then the callback passed to
+            // `fastify.close` will never be invoked; nor will any of the
+            // registered `onClose` hooks.
+            conn.destroy()
+            fastify[kKeepAliveConnections].delete(conn)
+          }
         }
       } else {
         done(null)
@@ -410,7 +433,8 @@ function fastify (options) {
     hasLogger,
     setupResponseListeners,
     throwIfAlreadyStarted,
-    validateHTTPVersion: compileValidateHTTPVersion(options)
+    validateHTTPVersion: compileValidateHTTPVersion(options),
+    keepAliveConnections
   })
 
   // Delay configuring clientError handler so that it can access fastify state.