fastify 4.5.3 → 4.7.0

package/test/internals/validation.test.js

@@ -268,6 +268,21 @@ test('build schema - headers are not lowercased in case of custom object', t =>
   })
 })
 
+test('build schema - headers are not lowercased in case of custom validator provided', t => {
+  t.plan(1)
+
+  class Headers {}
+  const opts = {
+    schema: {
+      headers: new Headers()
+    }
+  }
+  validation.compileSchemasForValidation(opts, ({ schema, method, url, httpPart }) => {
+    t.type(schema, Headers)
+    return () => {}
+  }, true)
+})
+
 test('build schema - uppercased headers are not included', t => {
   t.plan(1)
   const opts = {

package/test/listen.deprecated.test.js

@@ -200,3 +200,36 @@ test('listen when firstArg is { path: string(pipe) } and with backlog and callba
     t.equal(address, '\\\\.\\pipe\\testPipe3')
   })
 })
+
+test('listen accepts a port as string, and callback', t => {
+  t.plan(2)
+  const fastify = Fastify()
+  t.teardown(fastify.close.bind(fastify))
+  const port = 3000
+  fastify.listen(port.toString(), localhost, (err) => {
+    t.equal(fastify.server.address().port, port)
+    t.error(err)
+  })
+})
+
+test('listen accepts a port as string, address and callback', t => {
+  t.plan(3)
+  const fastify = Fastify()
+  t.teardown(fastify.close.bind(fastify))
+  const port = 3000
+  fastify.listen(port.toString(), localhost, (err) => {
+    t.equal(fastify.server.address().port, port)
+    t.equal(fastify.server.address().address, localhost)
+    t.error(err)
+  })
+})
+
+test('listen with invalid port string without callback with (address)', t => {
+  t.plan(1)
+  const fastify = Fastify()
+  t.teardown(fastify.close.bind(fastify))
+  fastify.listen('-1')
+    .then(address => {
+      t.equal(address, `http://${localhostForURL}:${fastify.server.address().port}`)
+    })
+})

package/test/plugin.test.js

@@ -885,7 +885,7 @@ test('pluginTimeout', t => {
   })
 })
 
-test('pluginTimeout - named function', { only: true }, t => {
+test('pluginTimeout - named function', t => {
   t.plan(5)
   const fastify = Fastify({
     pluginTimeout: 10

package/test/reply-code.test.js

@@ -0,0 +1,59 @@
+'use strict'
+
+const t = require('tap')
+const test = t.test
+const Fastify = require('..')
+
+test('code should handle null/undefined/float', t => {
+  t.plan(8)
+
+  const fastify = Fastify()
+
+  fastify.get('/null', function (request, reply) {
+    reply.status(null).send()
+  })
+
+  fastify.get('/undefined', function (request, reply) {
+    reply.status(undefined).send()
+  })
+
+  fastify.get('/404.5', function (request, reply) {
+    reply.status(404.5).send()
+  })
+
+  fastify.inject({
+    method: 'GET',
+    url: '/null'
+  }, (error, res) => {
+    t.error(error)
+    t.equal(res.statusCode, 500)
+    t.same(res.json(), {
+      statusCode: 500,
+      code: 'FST_ERR_BAD_STATUS_CODE',
+      error: 'Internal Server Error',
+      message: 'Called reply with an invalid status code: null'
+    })
+  })
+
+  fastify.inject({
+    method: 'GET',
+    url: '/undefined'
+  }, (error, res) => {
+    t.error(error)
+    t.equal(res.statusCode, 500)
+    t.same(res.json(), {
+      statusCode: 500,
+      code: 'FST_ERR_BAD_STATUS_CODE',
+      error: 'Internal Server Error',
+      message: 'Called reply with an invalid status code: undefined'
+    })
+  })
+
+  fastify.inject({
+    method: 'GET',
+    url: '/404.5'
+  }, (error, res) => {
+    t.error(error)
+    t.equal(res.statusCode, 404)
+  })
+})

package/test/route.test.js

@@ -1464,3 +1464,32 @@ test('invalid url attribute - non string URL', t => {
     t.equal(error.code, FST_ERR_INVALID_URL().code)
   }
 })
+
+test('exposeHeadRoute should not reuse the same route option', async t => {
+  t.plan(2)
+
+  const fastify = Fastify()
+
+  // we update the onRequest hook in onRoute hook
+  // if we reuse the same route option
+  // that means we will append another function inside the array
+  fastify.addHook('onRoute', function (routeOption) {
+    if (Array.isArray(routeOption.onRequest)) {
+      routeOption.onRequest.push(() => {})
+    } else {
+      routeOption.onRequest = [() => {}]
+    }
+  })
+
+  fastify.addHook('onRoute', function (routeOption) {
+    t.equal(routeOption.onRequest.length, 1)
+  })
+
+  fastify.route({
+    method: 'GET',
+    path: '/more-coffee',
+    async handler () {
+      return 'hello world'
+    }
+  })
+})

package/test/router-options.test.js

@@ -1,20 +1,10 @@
 'use strict'
 
 const test = require('tap').test
-const sget = require('simple-get')
 const Fastify = require('../')
 const { FST_ERR_BAD_URL } = require('../lib/errors')
 
-function getUrl (app) {
-  const { address, port } = app.server.address()
-  if (address === '::1') {
-    return `http://[${address}]:${port}`
-  } else {
-    return `http://${address}:${port}`
-  }
-}
-
-test('Should honor ignoreTrailingSlash option', t => {
+test('Should honor ignoreTrailingSlash option', async t => {
   t.plan(4)
   const fastify = Fastify({
     ignoreTrailingSlash: true
@@ -24,27 +14,16 @@ test('Should honor ignoreTrailingSlash option', t => {
     res.send('test')
   })
 
-  fastify.listen({ port: 0 }, (err) => {
-    t.teardown(() => { fastify.close() })
-    if (err) t.threw(err)
-
-    const baseUrl = getUrl(fastify)
-
-    sget.concat(baseUrl + '/test', (err, res, data) => {
-      if (err) t.threw(err)
-      t.equal(res.statusCode, 200)
-      t.equal(data.toString(), 'test')
-    })
+  let res = await fastify.inject('/test')
+  t.equal(res.statusCode, 200)
+  t.equal(res.payload.toString(), 'test')
 
-    sget.concat(baseUrl + '/test/', (err, res, data) => {
-      if (err) t.threw(err)
-      t.equal(res.statusCode, 200)
-      t.equal(data.toString(), 'test')
-    })
-  })
+  res = await fastify.inject('/test/')
+  t.equal(res.statusCode, 200)
+  t.equal(res.payload.toString(), 'test')
 })
 
-test('Should honor ignoreDuplicateSlashes option', t => {
+test('Should honor ignoreDuplicateSlashes option', async t => {
   t.plan(4)
   const fastify = Fastify({
     ignoreDuplicateSlashes: true
@@ -54,27 +33,16 @@ test('Should honor ignoreDuplicateSlashes option', t => {
     res.send('test')
   })
 
-  fastify.listen({ port: 0 }, (err) => {
-    t.teardown(() => { fastify.close() })
-    if (err) t.threw(err)
-
-    const baseUrl = getUrl(fastify)
-
-    sget.concat(baseUrl + '/test/test/test', (err, res, data) => {
-      if (err) t.threw(err)
-      t.equal(res.statusCode, 200)
-      t.equal(data.toString(), 'test')
-    })
+  let res = await fastify.inject('/test/test/test')
+  t.equal(res.statusCode, 200)
+  t.equal(res.payload.toString(), 'test')
 
-    sget.concat(baseUrl + '/test//test///test', (err, res, data) => {
-      if (err) t.threw(err)
-      t.equal(res.statusCode, 200)
-      t.equal(data.toString(), 'test')
-    })
-  })
+  res = await fastify.inject('/test//test///test')
+  t.equal(res.statusCode, 200)
+  t.equal(res.payload.toString(), 'test')
 })
 
-test('Should honor ignoreTrailingSlash and ignoreDuplicateSlashes options', t => {
+test('Should honor ignoreTrailingSlash and ignoreDuplicateSlashes options', async t => {
   t.plan(4)
   const fastify = Fastify({
     ignoreTrailingSlash: true,
@@ -85,24 +53,13 @@ test('Should honor ignoreTrailingSlash and ignoreDuplicateSlashes options', t =>
     res.send('test')
   })
 
-  fastify.listen({ port: 0 }, (err) => {
-    t.teardown(() => { fastify.close() })
-    if (err) t.threw(err)
-
-    const baseUrl = getUrl(fastify)
+  let res = await fastify.inject('/test/test/test/')
+  t.equal(res.statusCode, 200)
+  t.equal(res.payload.toString(), 'test')
 
-    sget.concat(baseUrl + '/test/test/test/', (err, res, data) => {
-      if (err) t.threw(err)
-      t.equal(res.statusCode, 200)
-      t.equal(data.toString(), 'test')
-    })
-
-    sget.concat(baseUrl + '/test//test///test//', (err, res, data) => {
-      if (err) t.threw(err)
-      t.equal(res.statusCode, 200)
-      t.equal(data.toString(), 'test')
-    })
-  })
+  res = await fastify.inject('/test//test///test//')
+  t.equal(res.statusCode, 200)
+  t.equal(res.payload.toString(), 'test')
 })
 
 test('Should honor maxParamLength option', t => {
@@ -131,12 +88,22 @@ test('Should honor maxParamLength option', t => {
 })
 
 test('Should expose router options via getters on request and reply', t => {
-  t.plan(7)
+  t.plan(10)
   const fastify = Fastify()
+  const expectedSchema = {
+    params: {
+      id: { type: 'integer' }
+    }
+  }
 
-  fastify.get('/test/:id', (req, reply) => {
+  fastify.get('/test/:id', {
+    schema: expectedSchema
+  }, (req, reply) => {
     t.equal(reply.context.config.url, '/test/:id')
     t.equal(reply.context.config.method, 'GET')
+    t.equal(req.routeConfig.url, '/test/:id')
+    t.equal(req.routeConfig.method, 'GET')
+    t.same(req.routeSchema, expectedSchema)
     t.equal(req.routerPath, '/test/:id')
     t.equal(req.routerMethod, 'GET')
     t.equal(req.is404, false)

package/test/schema-feature.test.js

@@ -1779,3 +1779,28 @@ test('Should return a human-friendly error if response status codes are not spec
     t.match(err.message, 'Failed building the serialization schema for GET: /, due to error response schemas should be nested under a valid status code, e.g { 2xx: { type: "object" } }')
   })
 })
+
+test('setSchemaController: custom validator instance should not mutate headers schema', async t => {
+  t.plan(2)
+  class Headers {}
+  const fastify = Fastify()
+
+  fastify.setSchemaController({
+    compilersFactory: {
+      buildValidator: function () {
+        return ({ schema, method, url, httpPart }) => {
+          t.type(schema, Headers)
+          return () => {}
+        }
+      }
+    }
+  })
+
+  fastify.get('/', {
+    schema: {
+      headers: new Headers()
+    }
+  }, () => {})
+
+  await fastify.ready()
+})

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

@@ -2,6 +2,7 @@
 
 const { test } = require('tap')
 const Joi = require('joi')
+const yup = require('yup')
 const AJV = require('ajv')
 const S = require('fluent-json-schema')
 const Fastify = require('..')
@@ -673,3 +674,31 @@ test('JOI validation overwrite request headers', t => {
     })
   })
 })
+
+test('Custom schema object should not trigger FST_ERR_SCH_DUPLICATE', async t => {
+  const fastify = Fastify()
+  const handler = () => { }
+
+  fastify.get('/the/url', {
+    schema: {
+      query: yup.object({
+        foo: yup.string()
+      })
+    },
+    validatorCompiler: ({ schema, method, url, httpPart }) => {
+      return function (data) {
+        // with option strict = false, yup `validateSync` function returns the coerced value if validation was successful, or throws if validation failed
+        try {
+          const result = schema.validateSync(data, {})
+          return { value: result }
+        } catch (e) {
+          return { error: e }
+        }
+      }
+    },
+    handler
+  })
+
+  await fastify.ready()
+  t.pass('fastify is ready')
+})

package/test/schema-validation.test.js

@@ -1017,3 +1017,22 @@ test("The same $id in route's schema must not overwrite others", t => {
     t.same(res.payload, 'ok')
   })
 })
+
+test('Custom validator compiler should not mutate schema', async t => {
+  t.plan(2)
+  class Headers {}
+  const fastify = Fastify()
+
+  fastify.setValidatorCompiler(({ schema, method, url, httpPart }) => {
+    t.type(schema, Headers)
+    return () => {}
+  })
+
+  fastify.get('/', {
+    schema: {
+      headers: new Headers()
+    }
+  }, () => {})
+
+  await fastify.ready()
+})

package/test/search.test.js

@@ -1,18 +1,17 @@
 'use strict'
 
 const t = require('tap')
+const sget = require('simple-get').concat
 const test = t.test
 const fastify = require('..')()
 
 const schema = {
-  schema: {
-    response: {
-      '2xx': {
-        type: 'object',
-        properties: {
-          hello: {
-            type: 'string'
-          }
+  response: {
+    '2xx': {
+      type: 'object',
+      properties: {
+        hello: {
+          type: 'string'
         }
       }
     }
@@ -20,35 +19,45 @@ const schema = {
 }
 
 const querySchema = {
-  schema: {
-    querystring: {
-      type: 'object',
-      properties: {
-        hello: {
-          type: 'integer'
-        }
+  querystring: {
+    type: 'object',
+    properties: {
+      hello: {
+        type: 'integer'
       }
     }
   }
 }
 
 const paramsSchema = {
-  schema: {
-    params: {
-      type: 'object',
-      properties: {
-        foo: {
-          type: 'string'
-        },
-        test: {
-          type: 'integer'
-        }
+  params: {
+    type: 'object',
+    properties: {
+      foo: {
+        type: 'string'
+      },
+      test: {
+        type: 'integer'
       }
     }
   }
 }
 
-test('shorthand - search', t => {
+const bodySchema = {
+  body: {
+    type: 'object',
+    properties: {
+      foo: {
+        type: 'string'
+      },
+      test: {
+        type: 'integer'
+      }
+    }
+  }
+}
+
+test('search', t => {
   t.plan(1)
   try {
     fastify.route({
@@ -65,13 +74,13 @@ test('shorthand - search', t => {
   }
 })
 
-test('shorthand - search params', t => {
+test('search, params schema', t => {
   t.plan(1)
   try {
     fastify.route({
       method: 'SEARCH',
       url: '/params/:foo/:test',
-      paramsSchema,
+      schema: paramsSchema,
       handler: function (request, reply) {
         reply.code(200).send(request.params)
       }
@@ -82,13 +91,13 @@ test('shorthand - search params', t => {
   }
 })
 
-test('shorthand - get, querystring schema', t => {
+test('search, querystring schema', t => {
   t.plan(1)
   try {
     fastify.route({
       method: 'SEARCH',
       url: '/query',
-      querySchema,
+      schema: querySchema,
       handler: function (request, reply) {
         reply.code(200).send(request.query)
       }
@@ -98,3 +107,133 @@ test('shorthand - get, querystring schema', t => {
     t.fail()
   }
 })
+
+test('search, body schema', t => {
+  t.plan(1)
+  try {
+    fastify.route({
+      method: 'SEARCH',
+      url: '/body',
+      schema: bodySchema,
+      handler: function (request, reply) {
+        reply.code(200).send(request.body)
+      }
+    })
+    t.pass()
+  } catch (e) {
+    t.fail()
+  }
+})
+
+fastify.listen({ port: 0 }, err => {
+  t.error(err)
+  t.teardown(() => { fastify.close() })
+
+  const url = `http://localhost:${fastify.server.address().port}`
+
+  test('request - search', t => {
+    t.plan(4)
+    sget({
+      method: 'SEARCH',
+      url
+    }, (err, response, body) => {
+      t.error(err)
+      t.equal(response.statusCode, 200)
+      t.equal(response.headers['content-length'], '' + body.length)
+      t.same(JSON.parse(body), { hello: 'world' })
+    })
+  })
+
+  test('request search params schema', t => {
+    t.plan(4)
+    sget({
+      method: 'SEARCH',
+      url: `${url}/params/world/123`
+    }, (err, response, body) => {
+      t.error(err)
+      t.equal(response.statusCode, 200)
+      t.equal(response.headers['content-length'], '' + body.length)
+      t.same(JSON.parse(body), { foo: 'world', test: 123 })
+    })
+  })
+
+  test('request search params schema error', t => {
+    t.plan(3)
+    sget({
+      method: 'SEARCH',
+      url: `${url}/params/world/string`
+    }, (err, response, body) => {
+      t.error(err)
+      t.equal(response.statusCode, 400)
+      t.same(JSON.parse(body), {
+        error: 'Bad Request',
+        message: 'params/test must be integer',
+        statusCode: 400
+      })
+    })
+  })
+
+  test('request search querystring schema', t => {
+    t.plan(4)
+    sget({
+      method: 'SEARCH',
+      url: `${url}/query?hello=123`
+    }, (err, response, body) => {
+      t.error(err)
+      t.equal(response.statusCode, 200)
+      t.equal(response.headers['content-length'], '' + body.length)
+      t.same(JSON.parse(body), { hello: 123 })
+    })
+  })
+
+  test('request search querystring schema error', t => {
+    t.plan(3)
+    sget({
+      method: 'SEARCH',
+      url: `${url}/query?hello=world`
+    }, (err, response, body) => {
+      t.error(err)
+      t.equal(response.statusCode, 400)
+      t.same(JSON.parse(body), {
+        error: 'Bad Request',
+        message: 'querystring/hello must be integer',
+        statusCode: 400
+      })
+    })
+  })
+
+  test('request search body schema', t => {
+    t.plan(4)
+    const replyBody = { foo: 'bar', test: 5 }
+    sget({
+      method: 'SEARCH',
+      url: `${url}/body`,
+      body: JSON.stringify(replyBody),
+      headers: { 'content-type': 'application/json' }
+    }, (err, response, body) => {
+      t.error(err)
+      t.equal(response.statusCode, 200)
+      t.equal(response.headers['content-length'], '' + body.length)
+      t.same(JSON.parse(body), replyBody)
+    })
+  })
+
+  test('request search body schema error', t => {
+    t.plan(4)
+    sget({
+      method: 'SEARCH',
+      url: `${url}/body`,
+      body: JSON.stringify({ foo: 'bar', test: 'test' }),
+      headers: { 'content-type': 'application/json' }
+    }, (err, response, body) => {
+      t.error(err)
+      t.equal(response.statusCode, 400)
+      t.equal(response.headers['content-length'], '' + body.length)
+      t.same(JSON.parse(body), {
+        error: 'Bad Request',
+        message: 'body/test must be integer',
+        statusCode: 400
+      })
+    })
+  })
+})