fastify 4.17.0 → 4.18.0

package/test/fastify-instance.test.js

@@ -3,6 +3,8 @@
 const t = require('tap')
 const test = t.test
 const Fastify = require('..')
+const os = require('os')
+
 const {
   kOptions,
   kErrorHandler
@@ -97,3 +99,40 @@ test('errorHandler in plugin should be separate from the external one', async t
   t.ok(fastify[kErrorHandler].func instanceof Function)
   t.same(fastify.errorHandler, fastify[kErrorHandler].func)
 })
+
+test('fastify instance should contains listeningOrigin property (with port and host)', async t => {
+  t.plan(1)
+  const port = 3000
+  const host = '127.0.0.1'
+  const fastify = Fastify()
+  await fastify.listen({ port, host })
+  t.same(fastify.listeningOrigin, `http://${host}:${port}`)
+  await fastify.close()
+})
+
+test('fastify instance should contains listeningOrigin property (with port and https)', async t => {
+  t.plan(1)
+  const port = 3000
+  const host = '127.0.0.1'
+  const fastify = Fastify({ https: {} })
+  await fastify.listen({ port, host })
+  t.same(fastify.listeningOrigin, `https://${host}:${port}`)
+  await fastify.close()
+})
+
+test('fastify instance should contains listeningOrigin property (no options)', async t => {
+  t.plan(1)
+  const fastify = Fastify()
+  await fastify.listen()
+  const address = fastify.server.address()
+  t.same(fastify.listeningOrigin, `http://${address.address}:${address.port}`)
+  await fastify.close()
+})
+
+test('fastify instance should contains listeningOrigin property (unix socket)', { skip: os.platform() === 'win32' }, async t => {
+  const fastify = Fastify()
+  const path = `fastify.${Date.now()}.sock`
+  await fastify.listen({ path })
+  t.same(fastify.listeningOrigin, path)
+  await fastify.close()
+})

package/test/hooks-async.test.js

@@ -194,6 +194,160 @@ test('preParsing hooks should be able to modify the payload', t => {
   })
 })
 
+test('preParsing hooks should be able to supply statusCode', t => {
+  t.plan(4)
+  const fastify = Fastify()
+
+  fastify.addHook('preParsing', async (req, reply, payload) => {
+    const stream = new Readable({
+      read () {
+        const error = new Error('kaboom')
+        error.statusCode = 408
+        this.destroy(error)
+      }
+    })
+    stream.receivedEncodedLength = 20
+    return stream
+  })
+
+  fastify.addHook('onError', async (req, res, err) => {
+    t.equal(err.statusCode, 408)
+  })
+
+  fastify.post('/', function (request, reply) {
+    t.fail('should not be called')
+  })
+
+  fastify.inject({
+    method: 'POST',
+    url: '/',
+    payload: { hello: 'world' }
+  }, (err, res) => {
+    t.error(err)
+    t.equal(res.statusCode, 408)
+    t.same(JSON.parse(res.payload), {
+      statusCode: 408,
+      error: 'Request Timeout',
+      message: 'kaboom'
+    })
+  })
+})
+
+test('preParsing hooks should ignore statusCode 200 in stream error', t => {
+  t.plan(4)
+  const fastify = Fastify()
+
+  fastify.addHook('preParsing', async (req, reply, payload) => {
+    const stream = new Readable({
+      read () {
+        const error = new Error('kaboom')
+        error.statusCode = 200
+        this.destroy(error)
+      }
+    })
+    stream.receivedEncodedLength = 20
+    return stream
+  })
+
+  fastify.addHook('onError', async (req, res, err) => {
+    t.equal(err.statusCode, 400)
+  })
+
+  fastify.post('/', function (request, reply) {
+    t.fail('should not be called')
+  })
+
+  fastify.inject({
+    method: 'POST',
+    url: '/',
+    payload: { hello: 'world' }
+  }, (err, res) => {
+    t.error(err)
+    t.equal(res.statusCode, 400)
+    t.same(JSON.parse(res.payload), {
+      statusCode: 400,
+      error: 'Bad Request',
+      message: 'kaboom'
+    })
+  })
+})
+
+test('preParsing hooks should ignore non-number statusCode in stream error', t => {
+  t.plan(4)
+  const fastify = Fastify()
+
+  fastify.addHook('preParsing', async (req, reply, payload) => {
+    const stream = new Readable({
+      read () {
+        const error = new Error('kaboom')
+        error.statusCode = '418'
+        this.destroy(error)
+      }
+    })
+    stream.receivedEncodedLength = 20
+    return stream
+  })
+
+  fastify.addHook('onError', async (req, res, err) => {
+    t.equal(err.statusCode, 400)
+  })
+
+  fastify.post('/', function (request, reply) {
+    t.fail('should not be called')
+  })
+
+  fastify.inject({
+    method: 'POST',
+    url: '/',
+    payload: { hello: 'world' }
+  }, (err, res) => {
+    t.error(err)
+    t.equal(res.statusCode, 400)
+    t.same(JSON.parse(res.payload), {
+      statusCode: 400,
+      error: 'Bad Request',
+      message: 'kaboom'
+    })
+  })
+})
+
+test('preParsing hooks should default to statusCode 400 if stream error', t => {
+  t.plan(4)
+  const fastify = Fastify()
+
+  fastify.addHook('preParsing', async (req, reply, payload) => {
+    const stream = new Readable({
+      read () {
+        this.destroy(new Error('kaboom'))
+      }
+    })
+    stream.receivedEncodedLength = 20
+    return stream
+  })
+
+  fastify.addHook('onError', async (req, res, err) => {
+    t.equal(err.statusCode, 400)
+  })
+
+  fastify.post('/', function (request, reply) {
+    t.fail('should not be called')
+  })
+
+  fastify.inject({
+    method: 'POST',
+    url: '/',
+    payload: { hello: 'world' }
+  }, (err, res) => {
+    t.error(err)
+    t.equal(res.statusCode, 400)
+    t.same(JSON.parse(res.payload), {
+      statusCode: 400,
+      error: 'Bad Request',
+      message: 'kaboom'
+    })
+  })
+})
+
 test('preParsing hooks should handle errors', t => {
   t.plan(3)
   const fastify = Fastify()

package/test/hooks.test.js

@@ -165,7 +165,7 @@ test('hooks', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port
+      url: 'http://127.0.0.1:' + fastify.server.address().port
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -175,7 +175,7 @@ test('hooks', t => {
 
     sget({
       method: 'HEAD',
-      url: 'http://localhost:' + fastify.server.address().port
+      url: 'http://127.0.0.1:' + fastify.server.address().port
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 500)
@@ -183,7 +183,7 @@ test('hooks', t => {
 
     sget({
       method: 'DELETE',
-      url: 'http://localhost:' + fastify.server.address().port
+      url: 'http://127.0.0.1:' + fastify.server.address().port
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 500)
@@ -288,7 +288,7 @@ test('onRequest hook should support encapsulation / 3', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/first'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -298,7 +298,7 @@ test('onRequest hook should support encapsulation / 3', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/second'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/second'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -349,7 +349,7 @@ test('preHandler hook should support encapsulation / 5', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/first'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -359,7 +359,7 @@ test('preHandler hook should support encapsulation / 5', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/second'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/second'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -966,7 +966,7 @@ test('onResponse hook should support encapsulation / 3', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/first'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -976,7 +976,7 @@ test('onResponse hook should support encapsulation / 3', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/second'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/second'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -1043,7 +1043,7 @@ test('onSend hook should support encapsulation / 2', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/first'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -1053,7 +1053,7 @@ test('onSend hook should support encapsulation / 2', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/second'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/second'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -1338,7 +1338,7 @@ test('onSend hook throws', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port
+      url: 'http://127.0.0.1:' + fastify.server.address().port
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -1347,21 +1347,21 @@ test('onSend hook throws', t => {
     })
     sget({
       method: 'POST',
-      url: 'http://localhost:' + fastify.server.address().port
+      url: 'http://127.0.0.1:' + fastify.server.address().port
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 500)
     })
     sget({
       method: 'DELETE',
-      url: 'http://localhost:' + fastify.server.address().port
+      url: 'http://127.0.0.1:' + fastify.server.address().port
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 500)
     })
     sget({
       method: 'PUT',
-      url: 'http://localhost:' + fastify.server.address().port
+      url: 'http://127.0.0.1:' + fastify.server.address().port
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 500)
@@ -2476,7 +2476,7 @@ test('preValidation hook should support encapsulation / 3', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/first'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -2486,7 +2486,7 @@ test('preValidation hook should support encapsulation / 3', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/second'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/second'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -2623,7 +2623,7 @@ test('preParsing hook should run before parsing and be able to modify the payloa
 
     sget({
       method: 'POST',
-      url: 'http://localhost:' + fastify.server.address().port + '/first',
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first',
       body: { hello: 'world' },
       json: true
     }, (err, response, body) => {
@@ -2666,7 +2666,7 @@ test('preParsing hooks should run in the order in which they are defined', t =>
 
     sget({
       method: 'POST',
-      url: 'http://localhost:' + fastify.server.address().port + '/first',
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first',
       body: { hello: 'world' },
       json: true
     }, (err, response, body) => {
@@ -2716,7 +2716,7 @@ test('preParsing hooks should support encapsulation', t => {
 
     sget({
       method: 'POST',
-      url: 'http://localhost:' + fastify.server.address().port + '/first',
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first',
       body: { hello: 'world' },
       json: true
     }, (err, response, body) => {
@@ -2728,7 +2728,7 @@ test('preParsing hooks should support encapsulation', t => {
 
     sget({
       method: 'POST',
-      url: 'http://localhost:' + fastify.server.address().port + '/second',
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/second',
       body: { hello: 'world' },
       json: true
     }, (err, response, body) => {
@@ -2837,7 +2837,7 @@ test('preParsing hook should support encapsulation / 3', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/first'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -2847,7 +2847,7 @@ test('preParsing hook should support encapsulation / 3', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/second'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/second'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -2899,7 +2899,7 @@ test('preSerialization hook should run before serialization and be able to modif
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/first'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -2950,7 +2950,7 @@ test('preSerialization hook should be able to throw errors which are validated a
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/first'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 500)
@@ -2984,7 +2984,7 @@ test('preSerialization hook which returned error should still run onError hooks'
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/first'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 500)
@@ -3018,7 +3018,7 @@ test('preSerialization hooks should run in the order in which they are defined',
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/first'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -3062,7 +3062,7 @@ test('preSerialization hooks should support encapsulation', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/first'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/first'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)
@@ -3072,7 +3072,7 @@ test('preSerialization hooks should support encapsulation', t => {
 
     sget({
       method: 'GET',
-      url: 'http://localhost:' + fastify.server.address().port + '/second'
+      url: 'http://127.0.0.1:' + fastify.server.address().port + '/second'
     }, (err, response, body) => {
       t.error(err)
       t.equal(response.statusCode, 200)