fastify 4.24.0 → 4.24.2

package/test/custom-parser.1.test.js

@@ -4,34 +4,8 @@ const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('../fastify')
-
 const jsonParser = require('fast-json-body')
-
-function plainTextParser (request, callback) {
-  let body = ''
-  request.setEncoding('utf8')
-  request.on('error', onError)
-  request.on('data', onData)
-  request.on('end', onEnd)
-  function onError (err) {
-    callback(err, null)
-  }
-  function onData (chunk) {
-    body += chunk
-  }
-  function onEnd () {
-    callback(null, body)
-  }
-}
-
-function getUrl (app) {
-  const { address, port } = app.server.address()
-  if (address === '::1') {
-    return `http://[${address}]:${port}`
-  } else {
-    return `http://${address}:${port}`
-  }
-}
+const { getServerUrl } = require('./helper')
 
 process.removeAllListeners('warning')
 
@@ -48,7 +22,7 @@ test('Should have typeof body object with no custom parser defined, null body an
 
     sget({
       method: 'POST',
-      url: getUrl(fastify),
+      url: getServerUrl(fastify),
       body: null,
       headers: {
         'Content-Type': 'text/plain'
@@ -75,7 +49,7 @@ test('Should have typeof body object with no custom parser defined, undefined bo
 
     sget({
       method: 'POST',
-      url: getUrl(fastify),
+      url: getServerUrl(fastify),
       body: undefined,
       headers: {
         'Content-Type': 'text/plain'
@@ -114,7 +88,7 @@ test('Should get the body as string', t => {
 
     sget({
       method: 'POST',
-      url: getUrl(fastify),
+      url: getServerUrl(fastify),
       body: 'hello world',
       headers: {
         'Content-Type': 'text/plain'
@@ -153,7 +127,7 @@ test('Should get the body as buffer', t => {
 
     sget({
       method: 'POST',
-      url: getUrl(fastify),
+      url: getServerUrl(fastify),
       body: '{"hello":"world"}',
       headers: {
         'Content-Type': 'application/json'
@@ -192,7 +166,7 @@ test('Should get the body as buffer', t => {
 
     sget({
       method: 'POST',
-      url: getUrl(fastify),
+      url: getServerUrl(fastify),
       body: 'hello world',
       headers: {
         'Content-Type': 'text/plain'
@@ -229,7 +203,7 @@ test('Should parse empty bodies as a string', t => {
 
     sget({
       method: 'POST',
-      url: getUrl(fastify),
+      url: getServerUrl(fastify),
       body: '',
       headers: {
         'Content-Type': 'text/plain'
@@ -242,7 +216,7 @@ test('Should parse empty bodies as a string', t => {
 
     sget({
       method: 'DELETE',
-      url: getUrl(fastify),
+      url: getServerUrl(fastify),
       body: '',
       headers: {
         'Content-Type': 'text/plain',
@@ -275,7 +249,7 @@ test('Should parse empty bodies as a buffer', t => {
 
     sget({
       method: 'POST',
-      url: getUrl(fastify),
+      url: getServerUrl(fastify),
       body: '',
       headers: {
         'Content-Type': 'text/plain'
@@ -309,7 +283,7 @@ test('The charset should not interfere with the content type handling', t => {
 
     sget({
       method: 'POST',
-      url: getUrl(fastify),
+      url: getServerUrl(fastify),
       body: '{"hello":"world"}',
       headers: {
         'Content-Type': 'application/json; charset=utf-8'
@@ -322,699 +296,3 @@ test('The charset should not interfere with the content type handling', t => {
     })
   })
 })
-
-test('Wrong parseAs parameter', t => {
-  t.plan(2)
-  const fastify = Fastify()
-
-  try {
-    fastify.addContentTypeParser('application/json', { parseAs: 'fireworks' }, () => {})
-    t.fail('should throw')
-  } catch (err) {
-    t.equal(err.code, 'FST_ERR_CTP_INVALID_PARSE_TYPE')
-    t.equal(err.message, "The body parser can only parse your data as 'string' or 'buffer', you asked 'fireworks' which is not supported.")
-  }
-})
-
-test('Should allow defining the bodyLimit per parser', t => {
-  t.plan(3)
-  const fastify = Fastify()
-  t.teardown(() => fastify.close())
-
-  fastify.post('/', (req, reply) => {
-    reply.send(req.body)
-  })
-
-  fastify.addContentTypeParser(
-    'x/foo',
-    { parseAs: 'string', bodyLimit: 5 },
-    function (req, body, done) {
-      t.fail('should not be invoked')
-      done()
-    }
-  )
-
-  fastify.listen({ port: 0 }, err => {
-    t.error(err)
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: '1234567890',
-      headers: {
-        'Content-Type': 'x/foo'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.strictSame(JSON.parse(body.toString()), {
-        statusCode: 413,
-        code: 'FST_ERR_CTP_BODY_TOO_LARGE',
-        error: 'Payload Too Large',
-        message: 'Request body is too large'
-      })
-      fastify.close()
-    })
-  })
-})
-
-test('route bodyLimit should take precedence over a custom parser bodyLimit', t => {
-  t.plan(3)
-  const fastify = Fastify()
-  t.teardown(() => fastify.close())
-
-  fastify.post('/', { bodyLimit: 5 }, (request, reply) => {
-    reply.send(request.body)
-  })
-
-  fastify.addContentTypeParser(
-    'x/foo',
-    { parseAs: 'string', bodyLimit: 100 },
-    function (req, body, done) {
-      t.fail('should not be invoked')
-      done()
-    }
-  )
-
-  fastify.listen({ port: 0 }, err => {
-    t.error(err)
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: '1234567890',
-      headers: { 'Content-Type': 'x/foo' }
-    }, (err, response, body) => {
-      t.error(err)
-      t.strictSame(JSON.parse(body.toString()), {
-        statusCode: 413,
-        code: 'FST_ERR_CTP_BODY_TOO_LARGE',
-        error: 'Payload Too Large',
-        message: 'Request body is too large'
-      })
-      fastify.close()
-    })
-  })
-})
-
-test('should be able to use default parser for extra content type', t => {
-  t.plan(4)
-  const fastify = Fastify()
-  t.teardown(() => fastify.close())
-
-  fastify.post('/', (request, reply) => {
-    reply.send(request.body)
-  })
-
-  fastify.addContentTypeParser('text/json', { parseAs: 'string' }, fastify.getDefaultJsonParser('ignore', 'ignore'))
-
-  fastify.listen({ port: 0 }, err => {
-    t.error(err)
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: '{"hello":"world"}',
-      headers: {
-        'Content-Type': 'text/json'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.strictSame(JSON.parse(body.toString()), { hello: 'world' })
-      fastify.close()
-    })
-  })
-})
-
-test('contentTypeParser should add a custom parser with RegExp value', t => {
-  t.plan(3)
-
-  const fastify = Fastify()
-
-  fastify.post('/', (req, reply) => {
-    reply.send(req.body)
-  })
-
-  fastify.options('/', (req, reply) => {
-    reply.send(req.body)
-  })
-
-  fastify.addContentTypeParser(/.*\+json$/, function (req, payload, done) {
-    jsonParser(payload, function (err, body) {
-      done(err, body)
-    })
-  })
-
-  fastify.listen({ port: 0 }, err => {
-    t.error(err)
-
-    t.teardown(() => fastify.close())
-
-    t.test('in POST', t => {
-      t.plan(3)
-
-      sget({
-        method: 'POST',
-        url: getUrl(fastify),
-        body: '{"hello":"world"}',
-        headers: {
-          'Content-Type': 'application/vnd.test+json'
-        }
-      }, (err, response, body) => {
-        t.error(err)
-        t.equal(response.statusCode, 200)
-        t.same(body.toString(), JSON.stringify({ hello: 'world' }))
-      })
-    })
-
-    t.test('in OPTIONS', t => {
-      t.plan(3)
-
-      sget({
-        method: 'OPTIONS',
-        url: getUrl(fastify),
-        body: '{"hello":"world"}',
-        headers: {
-          'Content-Type': 'weird/content-type+json'
-        }
-      }, (err, response, body) => {
-        t.error(err)
-        t.equal(response.statusCode, 200)
-        t.same(body.toString(), JSON.stringify({ hello: 'world' }))
-      })
-    })
-  })
-})
-
-test('contentTypeParser should add multiple custom parsers with RegExp values', async t => {
-  t.plan(6)
-  const fastify = Fastify()
-  t.teardown(fastify.close.bind(fastify))
-
-  fastify.post('/', (req, reply) => {
-    reply.send(req.body)
-  })
-
-  fastify.addContentTypeParser(/.*\+json$/, function (req, payload, done) {
-    jsonParser(payload, function (err, body) {
-      done(err, body)
-    })
-  })
-
-  fastify.addContentTypeParser(/.*\+xml$/, function (req, payload, done) {
-    done(null, 'xml')
-  })
-
-  fastify.addContentTypeParser(/.*\+myExtension$/i, function (req, payload, done) {
-    let data = ''
-    payload.on('data', chunk => { data += chunk })
-    payload.on('end', () => {
-      done(null, data + 'myExtension')
-    })
-  })
-
-  await fastify.ready()
-
-  {
-    const response = await fastify.inject({
-      method: 'POST',
-      url: '/',
-      body: '{"hello":"world"}',
-      headers: {
-        'Content-Type': 'application/vnd.hello+json'
-      }
-    })
-    t.equal(response.statusCode, 200)
-    t.same(response.payload.toString(), '{"hello":"world"}')
-  }
-
-  {
-    const response = await fastify.inject({
-      method: 'POST',
-      url: '/',
-      body: '{"hello":"world"}',
-      headers: {
-        'Content-Type': 'application/test+xml'
-      }
-    })
-    t.equal(response.statusCode, 200)
-    t.same(response.payload.toString(), 'xml')
-  }
-
-  await fastify.inject({
-    method: 'POST',
-    path: '/',
-    payload: 'abcdefg',
-    headers: {
-      'Content-Type': 'application/+myExtension'
-    }
-  }).then((response) => {
-    t.equal(response.statusCode, 200)
-    t.same(response.payload.toString(), 'abcdefgmyExtension')
-  }).catch((err) => {
-    t.error(err)
-  })
-})
-
-test('catch all content type parser should not interfere with content type parser', t => {
-  t.plan(10)
-  const fastify = Fastify()
-  t.teardown(fastify.close.bind(fastify))
-
-  fastify.post('/', (req, reply) => {
-    reply.send(req.body)
-  })
-
-  fastify.addContentTypeParser('*', function (req, payload, done) {
-    let data = ''
-    payload.on('data', chunk => { data += chunk })
-    payload.on('end', () => {
-      done(null, data)
-    })
-  })
-
-  fastify.addContentTypeParser(/^application\/.*/, function (req, payload, done) {
-    jsonParser(payload, function (err, body) {
-      done(err, body)
-    })
-  })
-
-  fastify.addContentTypeParser('text/html', function (req, payload, done) {
-    let data = ''
-    payload.on('data', chunk => { data += chunk })
-    payload.on('end', () => {
-      done(null, data + 'html')
-    })
-  })
-
-  fastify.listen({ port: 0 }, err => {
-    t.error(err)
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: '{"myKey":"myValue"}',
-      headers: {
-        'Content-Type': 'application/json'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.same(body.toString(), JSON.stringify({ myKey: 'myValue' }))
-    })
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: 'body',
-      headers: {
-        'Content-Type': 'very-weird-content-type'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.same(body.toString(), 'body')
-    })
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: 'my text',
-      headers: {
-        'Content-Type': 'text/html'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.same(body.toString(), 'my texthtml')
-    })
-  })
-})
-
-test('should prefer string content types over RegExp ones', t => {
-  t.plan(7)
-  const fastify = Fastify()
-  t.teardown(fastify.close.bind(fastify))
-
-  fastify.post('/', (req, reply) => {
-    reply.send(req.body)
-  })
-
-  fastify.addContentTypeParser(/^application\/.*/, function (req, payload, done) {
-    let data = ''
-    payload.on('data', chunk => { data += chunk })
-    payload.on('end', () => {
-      done(null, data)
-    })
-  })
-
-  fastify.addContentTypeParser('application/json', function (req, payload, done) {
-    jsonParser(payload, function (err, body) {
-      done(err, body)
-    })
-  })
-
-  fastify.listen({ port: 0 }, err => {
-    t.error(err)
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: '{"k1":"myValue", "k2": "myValue"}',
-      headers: {
-        'Content-Type': 'application/json'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.same(body.toString(), JSON.stringify({ k1: 'myValue', k2: 'myValue' }))
-    })
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: 'javascript',
-      headers: {
-        'Content-Type': 'application/javascript'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.same(body.toString(), 'javascript')
-    })
-  })
-})
-
-test('removeContentTypeParser should support arrays of content types to remove', t => {
-  t.plan(8)
-
-  const fastify = Fastify()
-  t.teardown(fastify.close.bind(fastify))
-
-  fastify.addContentTypeParser('application/xml', function (req, payload, done) {
-    payload.on('data', () => {})
-    payload.on('end', () => {
-      done(null, 'xml')
-    })
-  })
-
-  fastify.addContentTypeParser(/^image\/.*/, function (req, payload, done) {
-    payload.on('data', () => {})
-    payload.on('end', () => {
-      done(null, 'image')
-    })
-  })
-
-  fastify.removeContentTypeParser([/^image\/.*/, 'application/json'])
-
-  fastify.post('/', (req, reply) => {
-    reply.send(req.body)
-  })
-
-  fastify.listen({ port: 0 }, err => {
-    t.error(err)
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: '<?xml version="1.0">',
-      headers: {
-        'Content-Type': 'application/xml'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.same(body.toString(), 'xml')
-    })
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: '',
-      headers: {
-        'Content-Type': 'image/png'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 415)
-    })
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: '{test: "test"}',
-      headers: {
-        'Content-Type': 'application/json'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 415)
-    })
-  })
-})
-
-test('removeContentTypeParser should support encapsulation', t => {
-  t.plan(6)
-
-  const fastify = Fastify()
-
-  fastify.addContentTypeParser('application/xml', function (req, payload, done) {
-    payload.on('data', () => {})
-    payload.on('end', () => {
-      done(null, 'xml')
-    })
-  })
-
-  fastify.post('/', (req, reply) => {
-    reply.send(req.body)
-  })
-
-  fastify.register(function (instance, options, done) {
-    instance.removeContentTypeParser('application/xml')
-
-    instance.post('/encapsulated', (req, reply) => {
-      reply.send(req.body)
-    })
-
-    done()
-  })
-
-  fastify.listen({ port: 0 }, err => {
-    t.error(err)
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify) + '/encapsulated',
-      body: '<?xml version="1.0">',
-      headers: {
-        'Content-Type': 'application/xml'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 415)
-    })
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: '<?xml version="1.0">',
-      headers: {
-        'Content-Type': 'application/xml'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.same(body.toString(), 'xml')
-      fastify.close()
-    })
-  })
-})
-
-test('removeAllContentTypeParsers should support encapsulation', t => {
-  t.plan(6)
-
-  const fastify = Fastify()
-
-  fastify.post('/', (req, reply) => {
-    reply.send(req.body)
-  })
-
-  fastify.register(function (instance, options, done) {
-    instance.removeAllContentTypeParsers()
-
-    instance.post('/encapsulated', (req, reply) => {
-      reply.send(req.body)
-    })
-
-    done()
-  })
-
-  fastify.listen({ port: 0 }, err => {
-    t.error(err)
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify) + '/encapsulated',
-      body: '{}',
-      headers: {
-        'Content-Type': 'application/json'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 415)
-    })
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: '{"test":1}',
-      headers: {
-        'Content-Type': 'application/json'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.same(JSON.parse(body.toString()).test, 1)
-      fastify.close()
-    })
-  })
-})
-
-test('cannot remove all content type parsers after binding', t => {
-  t.plan(2)
-
-  const fastify = Fastify()
-
-  t.teardown(fastify.close.bind(fastify))
-
-  fastify.listen({ port: 0 }, function (err) {
-    t.error(err)
-
-    t.throws(() => fastify.removeAllContentTypeParsers())
-  })
-})
-
-test('cannot remove content type parsers after binding', t => {
-  t.plan(2)
-
-  const fastify = Fastify()
-
-  t.teardown(fastify.close.bind(fastify))
-
-  fastify.listen({ port: 0 }, function (err) {
-    t.error(err)
-
-    t.throws(() => fastify.removeContentTypeParser('application/json'))
-  })
-})
-
-test('should be able to override the default json parser after removeAllContentTypeParsers', t => {
-  t.plan(5)
-
-  const fastify = Fastify()
-
-  fastify.post('/', (req, reply) => {
-    reply.send(req.body)
-  })
-
-  fastify.removeAllContentTypeParsers()
-
-  fastify.addContentTypeParser('application/json', function (req, payload, done) {
-    t.ok('called')
-    jsonParser(payload, function (err, body) {
-      done(err, body)
-    })
-  })
-
-  fastify.listen({ port: 0 }, err => {
-    t.error(err)
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: '{"hello":"world"}',
-      headers: {
-        'Content-Type': 'application/json'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.same(body.toString(), JSON.stringify({ hello: 'world' }))
-      fastify.close()
-    })
-  })
-})
-
-test('should be able to override the default plain text parser after removeAllContentTypeParsers', t => {
-  t.plan(5)
-
-  const fastify = Fastify()
-
-  fastify.post('/', (req, reply) => {
-    reply.send(req.body)
-  })
-
-  fastify.removeAllContentTypeParsers()
-
-  fastify.addContentTypeParser('text/plain', function (req, payload, done) {
-    t.ok('called')
-    plainTextParser(payload, function (err, body) {
-      done(err, body)
-    })
-  })
-
-  fastify.listen({ port: 0 }, err => {
-    t.error(err)
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: 'hello world',
-      headers: {
-        'Content-Type': 'text/plain'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.equal(body.toString(), 'hello world')
-      fastify.close()
-    })
-  })
-})
-
-test('should be able to add a custom content type parser after removeAllContentTypeParsers', t => {
-  t.plan(5)
-
-  const fastify = Fastify()
-
-  fastify.post('/', (req, reply) => {
-    reply.send(req.body)
-  })
-
-  fastify.removeAllContentTypeParsers()
-
-  fastify.addContentTypeParser('application/jsoff', function (req, payload, done) {
-    t.ok('called')
-    jsonParser(payload, function (err, body) {
-      done(err, body)
-    })
-  })
-
-  fastify.listen({ port: 0 }, err => {
-    t.error(err)
-
-    sget({
-      method: 'POST',
-      url: getUrl(fastify),
-      body: '{"hello":"world"}',
-      headers: {
-        'Content-Type': 'application/jsoff'
-      }
-    }, (err, response, body) => {
-      t.error(err)
-      t.equal(response.statusCode, 200)
-      t.same(body.toString(), JSON.stringify({ hello: 'world' }))
-      fastify.close()
-    })
-  })
-})