fastify 4.0.0-alpha.1 → 4.0.0-alpha.2

package/test/logger.test.js

@@ -56,7 +56,7 @@ test('defaults to info level', t => {
     reply.send({ hello: 'world' })
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.server.unref()
 
@@ -118,7 +118,7 @@ test('test log stream', t => {
     reply.send({ hello: 'world' })
   })
 
-  fastify.listen(0, localhost, err => {
+  fastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     fastify.server.unref()
 
@@ -165,7 +165,7 @@ test('test error log stream', t => {
     reply.send(new Error('kaboom'))
   })
 
-  fastify.listen(0, localhost, err => {
+  fastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     fastify.server.unref()
 
@@ -210,7 +210,7 @@ test('can use external logger instance', t => {
     reply.send({ hello: 'world' })
   })
 
-  localFastify.listen(0, localhost, err => {
+  localFastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     http.get(`http://${localhostForURL}:` + localFastify.server.address().port + '/foo', (res) => {
       res.resume()
@@ -255,7 +255,7 @@ test('can use external logger instance with custom serializer', t => {
     reply.send({ hello: 'world' })
   })
 
-  localFastify.listen(0, localhost, err => {
+  localFastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     http.get(`http://${localhostForURL}:` + localFastify.server.address().port + '/foo', (res) => {
       res.resume()
@@ -459,7 +459,7 @@ test('The logger should accept custom serializer', t => {
     reply.send(new Error('kaboom'))
   })
 
-  fastify.listen(0, localhost, err => {
+  fastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     fastify.server.unref()
 
@@ -1175,7 +1175,7 @@ test('should serialize request and response', t => {
           level: 'info'
         }
       })
-      fastify.listen(0, ipv6, err => {
+      fastify.listen({ port: 0, host: ipv6 }, err => {
         t.error(err)
         stream.once('data', line => {
           const expected = 'Server listening at http://[' + ipv6 + ']:' +
@@ -1197,7 +1197,7 @@ test('Do not wrap IPv4 address', t => {
       level: 'info'
     }
   })
-  fastify.listen(0, '127.0.0.1', err => {
+  fastify.listen({ port: 0, host: '127.0.0.1' }, err => {
     t.error(err)
     stream.once('data', line => {
       const expected = 'Server listening at http://127.0.0.1:' +
@@ -1224,7 +1224,7 @@ test('file option', t => {
     reply.send({ hello: 'world' })
   })
 
-  fastify.listen(0, localhost, err => {
+  fastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     fastify.server.unref()
 
@@ -1269,7 +1269,7 @@ test('should log the error if no error handler is defined', t => {
     t.ok(req.log)
     reply.send(new Error('a generic error'))
   })
-  fastify.listen(0, localhost, err => {
+  fastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     fastify.server.unref()
     http.get(`http://${localhostForURL}:` + fastify.server.address().port + '/error')
@@ -1307,7 +1307,7 @@ test('should log as info if error status code >= 400 and < 500 if no error handl
     t.ok(req.log)
     reply.send(Object.assign(new Error('a 503 error'), { statusCode: 503 }))
   })
-  fastify.listen(0, localhost, err => {
+  fastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     fastify.server.unref()
     http.get(`http://${localhostForURL}:` + fastify.server.address().port + '/400')
@@ -1341,7 +1341,7 @@ test('should log as error if error status code >= 500 if no error handler is def
     t.ok(req.log)
     reply.send(Object.assign(new Error('a 503 error'), { statusCode: 503 }))
   })
-  fastify.listen(0, localhost, err => {
+  fastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     fastify.server.unref()
     http.get(`http://${localhostForURL}:` + fastify.server.address().port + '/503')
@@ -1379,7 +1379,7 @@ test('should not log the error if error handler is defined and it does not error
     t.ok(err)
     reply.send('something bad happened')
   })
-  fastify.listen(0, localhost, err => {
+  fastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     fastify.server.unref()
     http.get(`http://${localhostForURL}:` + fastify.server.address().port + '/error')
@@ -1410,7 +1410,7 @@ test('should not rely on raw request to log errors', t => {
     t.ok(req.log)
     reply.status(415).send(new Error('something happened'))
   })
-  fastify.listen(0, localhost, err => {
+  fastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     fastify.server.unref()
     http.get(`http://${localhostForURL}:` + fastify.server.address().port + '/error')
@@ -1460,7 +1460,7 @@ test('should redact the authorization header if so specified', t => {
       t.equal(line.req.headers.authorization, '[Redacted]', 'authorization is redacted')
     })
   })
-  fastify.listen(0, localhost, err => {
+  fastify.listen({ port: 0, host: localhost }, err => {
     t.error(err)
     fastify.server.unref()
     sget({

package/test/maxRequestsPerSocket.test.js

@@ -17,7 +17,7 @@ test('maxRequestsPerSocket on node version >= 16.10.0', { skip }, t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, function (err) {
+  fastify.listen({ port: 0 }, function (err) {
     t.error(err)
 
     const port = fastify.server.address().port
@@ -58,7 +58,7 @@ test('maxRequestsPerSocket zero should behave same as null', { skip }, t => {
 
   t.teardown(fastify.close.bind(fastify))
 
-  fastify.listen(0, function (err) {
+  fastify.listen({ port: 0 }, function (err) {
     t.error(err)
 
     const port = fastify.server.address().port

package/test/nullable-validation.test.js

@@ -88,7 +88,7 @@ test('object or null body', t => {
     }
   })
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     fastify.server.unref()
     t.error(err)
 
@@ -140,7 +140,7 @@ test('nullable body', t => {
     }
   })
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     fastify.server.unref()
     t.error(err)
 
@@ -181,7 +181,7 @@ test('Nullable body with 204', t => {
     }
   })
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     fastify.server.unref()
     t.error(err)
 

package/test/output-validation.test.js

@@ -90,7 +90,7 @@ test('unlisted response code', t => {
   }
 })
 
-fastify.listen(0, err => {
+fastify.listen({ port: 0 }, err => {
   t.error(err)
   fastify.server.unref()
 

package/test/plugin.test.js

@@ -89,7 +89,7 @@ test('fastify.register with fastify-plugin should not encapsulate his code', t =
     t.notOk(fastify.test)
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.server.unref()
 
@@ -168,7 +168,7 @@ test('fastify.register with fastify-plugin should provide access to external fas
     t.notOk(fastify.global)
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.server.unref()
 
@@ -223,7 +223,7 @@ test('fastify.register with fastify-plugin registers root level plugins', t => {
     reply.send({ test: fastify.test })
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.server.unref()
 
@@ -285,7 +285,7 @@ test('check dependencies - should not throw', t => {
     t.notOk(fastify.otherTest)
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.server.unref()
 
@@ -337,7 +337,7 @@ test('check dependencies - should throw', t => {
     t.notOk(fastify.test)
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.server.unref()
 
@@ -375,7 +375,7 @@ test('set the plugin name based on the plugin displayName symbol', t => {
     done()
   }, { name: 'plugin-B' }))
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.close()
   })
@@ -405,7 +405,7 @@ test('plugin name will change when using no encapsulation', t => {
     done()
   }, { name: 'plugin-A' }))
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.close()
   })
@@ -417,7 +417,7 @@ test('plugin name is undefined when accessing in no plugin context', t => {
 
   t.equal(fastify.pluginName, undefined)
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.close()
   })
@@ -445,7 +445,7 @@ test('set the plugin name based on the plugin function name', t => {
     done()
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.close()
   })
@@ -472,7 +472,7 @@ test('approximate a plugin name when no meta data is available', t => {
     done()
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.close()
   })
@@ -495,7 +495,7 @@ test('approximate a plugin name also when fastify-plugin has no meta data', t =>
     done()
   }))
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.close()
   })
@@ -535,7 +535,7 @@ test('plugin encapsulation', t => {
     t.notOk(fastify.test)
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.server.unref()
 
@@ -569,7 +569,7 @@ test('if a plugin raises an error and there is not a callback to handle it, the
     done(new Error('err'))
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.ok(err instanceof Error)
     t.equal(err.message, 'err')
   })
@@ -612,7 +612,7 @@ test('add hooks after route declaration', t => {
     done()
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     sget({
@@ -651,7 +651,7 @@ test('nested plugins', t => {
     done()
   }, { prefix: '/parent' })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     sget({
@@ -693,7 +693,7 @@ test('nested plugins awaited', t => {
     }, { prefix: '/child2' })
   }, { prefix: '/parent' })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     sget({

package/test/promises.test.js

@@ -60,7 +60,7 @@ fastify.get('/return-reply', opts, function (req, reply) {
   return reply.send({ hello: 'world' })
 })
 
-fastify.listen(0, err => {
+fastify.listen({ port: 0 }, err => {
   t.error(err)
   fastify.server.unref()
 

package/test/proto-poisoning.test.js

@@ -15,7 +15,7 @@ test('proto-poisoning error', t => {
     t.fail('handler should not be called')
   })
 
-  fastify.listen(0, function (err) {
+  fastify.listen({ port: 0 }, function (err) {
     t.error(err)
 
     sget({
@@ -41,7 +41,7 @@ test('proto-poisoning remove', t => {
     reply.send({ ok: true })
   })
 
-  fastify.listen(0, function (err) {
+  fastify.listen({ port: 0 }, function (err) {
     t.error(err)
 
     sget({
@@ -67,7 +67,7 @@ test('proto-poisoning ignore', t => {
     reply.send({ ok: true })
   })
 
-  fastify.listen(0, function (err) {
+  fastify.listen({ port: 0 }, function (err) {
     t.error(err)
 
     sget({
@@ -92,7 +92,7 @@ test('constructor-poisoning error (default in v3)', t => {
     reply.send('ok')
   })
 
-  fastify.listen(0, function (err) {
+  fastify.listen({ port: 0 }, function (err) {
     t.error(err)
 
     sget({
@@ -117,7 +117,7 @@ test('constructor-poisoning error', t => {
     t.fail('handler should not be called')
   })
 
-  fastify.listen(0, function (err) {
+  fastify.listen({ port: 0 }, function (err) {
     t.error(err)
 
     sget({
@@ -143,7 +143,7 @@ test('constructor-poisoning remove', t => {
     reply.send({ ok: true })
   })
 
-  fastify.listen(0, function (err) {
+  fastify.listen({ port: 0 }, function (err) {
     t.error(err)
 
     sget({

package/test/register.test.js

@@ -38,7 +38,7 @@ test('register', t => {
     done()
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.server.unref()
 
@@ -71,7 +71,7 @@ test('internal route declaration should pass the error generated by the register
     reply.send({ hello: 'world' })
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     fastify.close()
     t.equal(err.message, 'kaboom')
   })
@@ -93,7 +93,7 @@ test('internal route declaration should pass the error generated by the register
     t.equal(err.message, 'kaboom')
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     fastify.close()
     t.error(err)
   })

package/test/reply-error.test.js

@@ -107,7 +107,7 @@ test('Should reply 400 on client error', t => {
   t.plan(2)
 
   const fastify = Fastify()
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     const client = net.connect(fastify.server.address().port)
@@ -156,7 +156,7 @@ test('Should set the response from client error handler', t => {
     }
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
 
     const client = net.connect(fastify.server.address().port)
@@ -671,7 +671,7 @@ test('setting content-type on reply object should not hang the server case 2', a
   })
 
   try {
-    await fastify.listen(0)
+    await fastify.listen({ port: 0 })
     const res = await fastify.inject({
       url: '/',
       method: 'GET'

package/test/request-error.test.js

@@ -105,7 +105,7 @@ test('default clientError handler ignores ECONNRESET', t => {
     })
   })
 
-  fastify.listen(0, function (err) {
+  fastify.listen({ port: 0 }, function (err) {
     t.error(err)
     fastify.server.unref()
 

package/test/route-hooks.test.js

@@ -510,7 +510,7 @@ test('onTimeout on route', t => {
     }
   })
 
-  fastify.listen(0, (err, address) => {
+  fastify.listen({ port: 0 }, (err, address) => {
     t.error(err)
     t.teardown(() => fastify.close())
 

package/test/route.test.js

@@ -113,7 +113,7 @@ test('route', t => {
     }
   })
 
-  fastify.listen(0, function (err) {
+  fastify.listen({ port: 0 }, function (err) {
     if (err) t.error(err)
     fastify.server.unref()
 
@@ -1005,7 +1005,7 @@ test('no warning for exposeHeadRoute', async t => {
 
   process.on('warning', listener)
 
-  await fastify.listen(0)
+  await fastify.listen({ port: 0 })
 
   process.removeListener('warning', listener)
 
@@ -1439,7 +1439,7 @@ test('route with non-english characters', t => {
     reply.send('here /föö')
   })
 
-  fastify.listen(0, err => {
+  fastify.listen({ port: 0 }, err => {
     t.error(err)
     fastify.server.unref()
 

package/test/router-options.test.js

@@ -24,7 +24,7 @@ test('Should honor ignoreTrailingSlash option', t => {
     res.send('test')
   })
 
-  fastify.listen(0, (err) => {
+  fastify.listen({ port: 0 }, (err) => {
     fastify.server.unref()
     if (err) t.threw(err)
 

package/test/schema-feature.test.js

@@ -23,7 +23,7 @@ const echoBody = (req, reply) => { reply.send(req.body) }
     t.plan(2)
     const fastify = Fastify()
     t.teardown(fastify.close.bind(fastify))
-    fastify.listen(0, err => {
+    fastify.listen({ port: 0 }, err => {
       t.error(err)
       try {
         fastify[f](() => { })

package/test/schema-special-usage.test.js

@@ -8,87 +8,6 @@ const Fastify = require('..')
 const ajvMergePatch = require('ajv-merge-patch')
 const ajvErrors = require('ajv-errors')
 
-const buildValidatorAJV6 = require('@fastify/ajv-compiler-6')
-
-test('Ajv6 usage instead of the bundle one', t => {
-  t.plan(2)
-
-  t.test('use old ajv6 option', t => {
-    t.plan(2)
-    const fastify = Fastify({
-      ajv: {
-        customOptions: { jsonPointers: true }
-      },
-      schemaController: {
-        compilersFactory: {
-          buildValidator: buildValidatorAJV6()
-        }
-      }
-    })
-
-    fastify.post('/', {
-      schema: {
-        body: {
-          type: 'object',
-          required: ['missing'],
-          properties: {
-            foo: {
-              type: 'string'
-            }
-          }
-        }
-      },
-      handler (req, reply) { reply.send({ ok: 1 }) }
-    })
-
-    fastify.ready(err => {
-      t.error(err)
-      t.pass('the removed ajv6 option let the startup pass')
-    })
-  })
-
-  t.test('use old ajv6 option within a response schema', t => {
-    t.plan(2)
-    const fastify = Fastify({
-      schemaController: {
-        compilersFactory: {
-          buildValidator: buildValidatorAJV6()
-        }
-      }
-    })
-
-    fastify.post('/', {
-      schema: {
-        body: {
-          type: 'object',
-          required: ['missing'],
-          properties: {
-            foo: {
-              type: 'string'
-            }
-          }
-        },
-        response: {
-          '2xx': {
-            type: 'object',
-            properties: {
-              ok: {
-                type: 'integer'
-              }
-            }
-          }
-        }
-      },
-      handler (req, reply) { reply.send({ ok: 1 }) }
-    })
-
-    fastify.ready(err => {
-      t.error(err)
-      t.pass('startup successful')
-    })
-  })
-})
-
 test('Ajv plugins array parameter', t => {
   t.plan(3)
   const fastify = Fastify({

package/test/skip-reply-send.test.js

@@ -201,7 +201,7 @@ function testHandlerOrBeforeHandlerHook (test, hookOrHandler) {
 
       nextHooks.forEach(h => app.addHook(h, async (req, reply) => t.fail(`${h} should not be called`)))
 
-      app.listen(0, err => {
+      app.listen({ port: 0 }, err => {
         t.error(err)
         const client = net.createConnection({ port: (app.server.address()).port }, () => {
           client.write('GET / HTTP/1.1\r\n\r\n')