fastify 3.27.4 → 4.0.0-alpha.3

package/test/http2/unknown-http-method.test.js

@@ -6,21 +6,15 @@ const Fastify = require('../..')
 const h2url = require('h2url')
 const msg = { hello: 'world' }
 
-let fastify
-try {
-  fastify = Fastify({
-    http2: true
-  })
-  t.pass('http2 successfully loaded')
-} catch (e) {
-  t.fail('http2 loading failed', e)
-}
+const fastify = Fastify({
+  http2: true
+})
 
 fastify.get('/', function (req, reply) {
   reply.code(200).send(msg)
 })
 
-fastify.listen(0, err => {
+fastify.listen({ port: 0 }, err => {
   t.error(err)
   fastify.server.unref()
 

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

@@ -5,15 +5,18 @@ const test = t.test
 const Fastify = require('../..')
 const https = require('https')
 const sget = require('simple-get').concat
+const dns = require('dns').promises
 
 const { buildCertificate } = require('../build-certificate')
 t.before(buildCertificate)
 
-test('Should support a custom https server', t => {
-  t.plan(6)
+test('Should support a custom https server', async t => {
+  const localAddresses = await dns.lookup('localhost', { all: true })
+
+  t.plan(localAddresses.length + 3)
 
   const serverFactory = (handler, opts) => {
-    t.ok(opts.serverFactory)
+    t.ok(opts.serverFactory, 'it is called twice for every HOST interface')
 
     const options = {
       key: global.context.key,
@@ -37,17 +40,20 @@ test('Should support a custom https server', t => {
     reply.send({ hello: 'world' })
   })
 
-  fastify.listen(0, err => {
-    t.error(err)
+  await fastify.listen({ port: 0 })
 
+  await new Promise((resolve, reject) => {
     sget({
       method: 'GET',
       url: 'https://localhost:' + fastify.server.address().port,
       rejectUnauthorized: false
     }, (err, response, body) => {
-      t.error(err)
+      if (err) {
+        return reject(err)
+      }
       t.equal(response.statusCode, 200)
       t.same(JSON.parse(body), { hello: 'world' })
+      resolve()
     })
   })
 })

package/test/https/https.test.js

@@ -32,7 +32,7 @@ test('https', (t) => {
     reply.code(200).send({ proto: req.protocol })
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.server.unref()
 

package/test/input-validation.js

@@ -3,7 +3,7 @@
 
 const sget = require('simple-get').concat
 const Ajv = require('ajv')
-const Joi = require('@hapi/joi')
+const Joi = require('joi')
 const yup = require('yup')
 
 module.exports.payloadMethod = function (method, t) {
@@ -134,7 +134,7 @@ module.exports.payloadMethod = function (method, t) {
     }
   })
 
-  fastify.listen(0, function (err) {
+  fastify.listen({ port: 0 }, function (err) {
     if (err) {
       t.error(err)
     }
@@ -182,7 +182,7 @@ module.exports.payloadMethod = function (method, t) {
         t.equal(response.statusCode, 400)
         t.same(body, {
           error: 'Bad Request',
-          message: 'body.hello should be integer',
+          message: 'body/hello must be integer',
           statusCode: 400
         })
       })

package/test/internals/handleRequest.test.js

@@ -1,7 +1,6 @@
 'use strict'
 
 const { test } = require('tap')
-const semver = require('semver')
 const handleRequest = require('../../lib/handleRequest')
 const internals = require('../../lib/handleRequest')[Symbol.for('internals')]
 const Request = require('../../lib/request')
@@ -40,13 +39,8 @@ test('handleRequest function - invoke with error', t => {
 })
 
 test('handler function - invalid schema', t => {
-  t.plan(2)
+  t.plan(1)
   const res = {}
-  res.end = () => {
-    t.equal(res.statusCode, 400)
-    t.pass()
-  }
-  res.writeHead = () => {}
   res.log = { error: () => {}, info: () => {} }
   const context = {
     config: {
@@ -61,6 +55,7 @@ test('handler function - invalid schema', t => {
         }
       }
     },
+    errorHandler: { func: () => { t.pass('errorHandler called') } },
     handler: () => {},
     Reply,
     Request,
@@ -108,39 +103,7 @@ test('handler function - preValidationCallback with finished response', t => {
   t.plan(0)
   const res = {}
   // Be sure to check only `writableEnded` where is available
-  if (semver.gte(process.versions.node, '12.9.0')) {
-    res.writableEnded = true
-  } else {
-    res.writable = false
-    res.finished = true
-  }
-  res.end = () => {
-    t.fail()
-  }
-  res.writeHead = () => {}
-  const context = {
-    handler: (req, reply) => {
-      t.fail()
-      reply.send(undefined)
-    },
-    Reply,
-    Request,
-    preValidation: null,
-    preHandler: [],
-    onSend: [],
-    onError: []
-  }
-  buildSchema(context, schemaValidator)
-  internals.handler({}, new Reply(res, { context }))
-})
-
-test('handler function - preValidationCallback with finished response (< v12.9.0)', t => {
-  t.plan(0)
-  const res = {}
-  // Be sure to check only `writableEnded` where is available
-  res.writable = false
-  res.finished = true
-
+  res.writableEnded = true
   res.end = () => {
     t.fail()
   }
@@ -176,7 +139,7 @@ test('request should be defined in onSend Hook on post request with content type
   fastify.post('/', (request, reply) => {
     reply.send(200)
   })
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     fastify.server.unref()
     t.error(err)
     sget({
@@ -207,7 +170,7 @@ test('request should be defined in onSend Hook on post request with content type
   fastify.post('/', (request, reply) => {
     reply.send(200)
   })
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     fastify.server.unref()
     t.error(err)
     sget({
@@ -238,7 +201,7 @@ test('request should be defined in onSend Hook on options request with content t
   fastify.options('/', (request, reply) => {
     reply.send(200)
   })
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     fastify.server.unref()
     t.error(err)
     sget({

package/test/internals/initialConfig.test.js

@@ -2,6 +2,7 @@
 
 const { test, before } = require('tap')
 const Fastify = require('../..')
+const helper = require('../helper')
 const http = require('http')
 const pino = require('pino')
 const split = require('split2')
@@ -9,10 +10,17 @@ const deepClone = require('rfdc')({ circles: true, proto: false })
 const { deepFreezeObject } = require('../../lib/initialConfigValidation').utils
 
 const { buildCertificate } = require('../build-certificate')
-before(buildCertificate)
 
 process.removeAllListeners('warning')
 
+let localhost
+let localhostForURL
+
+before(async function () {
+  await buildCertificate();
+  [localhost, localhostForURL] = await helper.getLoopbackHost()
+})
+
 test('Fastify.initialConfig is an object', t => {
   t.plan(1)
   t.type(Fastify().initialConfig, 'object')
@@ -23,7 +31,8 @@ test('without options passed to Fastify, initialConfig should expose default val
 
   const fastifyDefaultOptions = {
     connectionTimeout: 0,
-    keepAliveTimeout: 5000,
+    keepAliveTimeout: 72000,
+    forceCloseConnections: false,
     maxRequestsPerSocket: 0,
     requestTimeout: 0,
     bodyLimit: 1024 * 1024,
@@ -37,7 +46,8 @@ test('without options passed to Fastify, initialConfig should expose default val
     pluginTimeout: 10000,
     requestIdHeader: 'request-id',
     requestIdLogLabel: 'reqId',
-    http2SessionTimeout: 5000
+    http2SessionTimeout: 72000,
+    exposeHeadRoutes: true
   }
 
   t.same(Fastify().initialConfig, fastifyDefaultOptions)
@@ -81,7 +91,7 @@ test('Fastify.initialConfig should expose all options', t => {
     ignoreTrailingSlash: true,
     maxParamLength: 200,
     connectionTimeout: 0,
-    keepAliveTimeout: 5000,
+    keepAliveTimeout: 72000,
     bodyLimit: 1049600,
     onProtoPoisoning: 'remove',
     serverFactory,
@@ -103,11 +113,11 @@ test('Fastify.initialConfig should expose all options', t => {
 
   const fastify = Fastify(options)
   t.equal(fastify.initialConfig.http2, true)
-  t.equal(fastify.initialConfig.https, true)
+  t.equal(fastify.initialConfig.https, true, 'for security reason the key cert is hidden')
   t.equal(fastify.initialConfig.ignoreTrailingSlash, true)
   t.equal(fastify.initialConfig.maxParamLength, 200)
   t.equal(fastify.initialConfig.connectionTimeout, 0)
-  t.equal(fastify.initialConfig.keepAliveTimeout, 5000)
+  t.equal(fastify.initialConfig.keepAliveTimeout, 72000)
   t.equal(fastify.initialConfig.bodyLimit, 1049600)
   t.equal(fastify.initialConfig.onProtoPoisoning, 'remove')
   t.equal(fastify.initialConfig.caseSensitive, true)
@@ -157,10 +167,19 @@ test('We must avoid shallow freezing and ensure that the whole object is freezed
     t.type(error, TypeError)
     t.equal(error.message, "Cannot assign to read only property 'allowHTTP1' of object '#<Object>'")
     t.ok(error.stack)
-    t.pass()
+    t.same(fastify.initialConfig.https, {
+      allowHTTP1: true
+    }, 'key cert removed')
   }
 })
 
+test('https value check', t => {
+  t.plan(1)
+
+  const fastify = Fastify({})
+  t.notOk(fastify.initialConfig.https)
+})
+
 test('Return an error if options do not match the validation schema', t => {
   t.plan(6)
 
@@ -171,7 +190,7 @@ test('Return an error if options do not match the validation schema', t => {
   } catch (error) {
     t.type(error, Error)
     t.equal(error.name, 'FastifyError')
-    t.equal(error.message, 'Invalid initialization options: \'["should be boolean"]\'')
+    t.equal(error.message, 'Invalid initialization options: \'["must be boolean"]\'')
     t.equal(error.code, 'FST_ERR_INIT_OPTS_INVALID')
     t.ok(error.stack)
     t.pass()
@@ -241,7 +260,8 @@ test('Should not have issues when passing stream options to Pino.js', t => {
     t.type(fastify, 'object')
     t.same(fastify.initialConfig, {
       connectionTimeout: 0,
-      keepAliveTimeout: 5000,
+      keepAliveTimeout: 72000,
+      forceCloseConnections: false,
       maxRequestsPerSocket: 0,
       requestTimeout: 0,
       bodyLimit: 1024 * 1024,
@@ -255,7 +275,8 @@ test('Should not have issues when passing stream options to Pino.js', t => {
       pluginTimeout: 10000,
       requestIdHeader: 'request-id',
       requestIdLogLabel: 'reqId',
-      http2SessionTimeout: 5000
+      http2SessionTimeout: 72000,
+      exposeHeadRoutes: true
     })
   } catch (error) {
     t.fail()
@@ -287,11 +308,11 @@ test('Should not have issues when passing stream options to Pino.js', t => {
     })
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     fastify.server.unref()
 
-    http.get('http://localhost:' + fastify.server.address().port)
+    http.get(`http://${localhostForURL}:${fastify.server.address().port}`)
   })
 })
 
@@ -356,3 +377,11 @@ test('Fastify.initialConfig should accept the deprecated versioning option', t =
     t.end()
   })
 })
+
+test('pluginTimeout should be parsed correctly', t => {
+  const withDisabledTimeout = Fastify({ pluginTimeout: '0' })
+  t.equal(withDisabledTimeout.initialConfig.pluginTimeout, 0)
+  const withInvalidTimeout = Fastify({ pluginTimeout: undefined })
+  t.equal(withInvalidTimeout.initialConfig.pluginTimeout, 10000)
+  t.end()
+})

package/test/internals/logger.test.js

@@ -20,7 +20,7 @@ test('The logger should add a unique id for every request', t => {
     reply.send({ id: req.id })
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     const queue = new Queue()
     for (let i = 0; i < 10; i++) {
@@ -53,7 +53,7 @@ test('The logger should reuse request id header for req.id', t => {
     reply.send({ id: req.id })
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     fastify.inject({