node-fastify 5.8.3

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

@@ -0,0 +1,218 @@
+'use strict'
+
+const { test } = require('node:test')
+const Fastify = require('../fastify')
+const jsonParser = require('fast-json-body')
+
+process.removeAllListeners('warning')
+
+test('should prefer string content types over RegExp ones', async (t) => {
+  t.plan(6)
+  const fastify = Fastify()
+  t.after(() => { fastify.close() })
+  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)
+    })
+  })
+
+  const fastifyServer = await fastify.listen({ port: 0 })
+
+  const result1 = await fetch(fastifyServer, {
+    method: 'POST',
+    body: '{"k1":"myValue", "k2": "myValue"}',
+    headers: {
+      'Content-Type': 'application/json'
+    }
+  })
+
+  t.assert.ok(result1.ok)
+  t.assert.strictEqual(result1.status, 200)
+  t.assert.equal(await result1.text(), JSON.stringify({ k1: 'myValue', k2: 'myValue' }))
+
+  const result2 = await fetch(fastifyServer, {
+    method: 'POST',
+    body: 'javascript',
+    headers: {
+      'Content-Type': 'application/javascript'
+    }
+  })
+
+  t.assert.ok(result2.ok)
+  t.assert.strictEqual(result2.status, 200)
+  t.assert.equal(await result2.text(), 'javascript')
+})
+
+test('removeContentTypeParser should support arrays of content types to remove', async (t) => {
+  t.plan(7)
+
+  const fastify = Fastify()
+  t.after(() => fastify.close())
+
+  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)
+  })
+
+  const fastifyServer = await fastify.listen({ port: 0 })
+
+  const result1 = await fetch(fastifyServer, {
+    method: 'POST',
+    body: '<?xml version="1.0">',
+    headers: {
+      'Content-Type': 'application/xml'
+    }
+  })
+
+  t.assert.ok(result1.ok)
+  t.assert.strictEqual(result1.status, 200)
+  t.assert.equal(await result1.text(), 'xml')
+
+  const result2 = await fetch(fastifyServer, {
+    method: 'POST',
+    body: '',
+    headers: {
+      'Content-Type': 'image/png'
+    }
+  })
+
+  t.assert.ok(!result2.ok)
+  t.assert.strictEqual(result2.status, 415)
+
+  const result3 = await fetch(fastifyServer, {
+    method: 'POST',
+    body: '{test: "test"}',
+    headers: {
+      'Content-Type': 'application/json'
+    }
+  })
+
+  t.assert.ok(!result3.ok)
+  t.assert.strictEqual(result3.status, 415)
+})
+
+test('removeContentTypeParser should support encapsulation', async (t) => {
+  t.plan(5)
+
+  const fastify = Fastify()
+  t.after(() => fastify.close())
+
+  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()
+  })
+
+  const fastifyServer = await fastify.listen({ port: 0 })
+
+  const result1 = await fetch(fastifyServer + '/encapsulated', {
+    method: 'POST',
+    body: '<?xml version="1.0">',
+    headers: {
+      'Content-Type': 'application/xml'
+    }
+  })
+
+  t.assert.ok(!result1.ok)
+  t.assert.strictEqual(result1.status, 415)
+
+  const result2 = await fetch(fastifyServer, {
+    method: 'POST',
+    body: '<?xml version="1.0">',
+    headers: {
+      'Content-Type': 'application/xml'
+    }
+  })
+
+  t.assert.ok(result2.ok)
+  t.assert.strictEqual(result2.status, 200)
+  t.assert.equal(await result2.text(), 'xml')
+})
+
+test('removeAllContentTypeParsers should support encapsulation', async (t) => {
+  t.plan(5)
+
+  const fastify = Fastify()
+  t.after(() => fastify.close())
+
+  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()
+  })
+
+  const fastifyServer = await fastify.listen({ port: 0 })
+
+  const result1 = await fetch(fastifyServer + '/encapsulated', {
+    method: 'POST',
+    body: '{}',
+    headers: {
+      'Content-Type': 'application/json'
+    }
+  })
+
+  t.assert.ok(!result1.ok)
+  t.assert.strictEqual(result1.status, 415)
+
+  const result2 = await fetch(fastifyServer, {
+    method: 'POST',
+    body: '{"test":1}',
+    headers: {
+      'Content-Type': 'application/json'
+    }
+  })
+
+  t.assert.ok(result2.ok)
+  t.assert.strictEqual(result2.status, 200)
+  t.assert.equal(JSON.parse(await result2.text()).test, 1)
+})

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

@@ -0,0 +1,130 @@
+'use strict'
+
+const { test } = require('node:test')
+const Fastify = require('../fastify')
+const jsonParser = require('fast-json-body')
+const { plainTextParser } = require('./helper')
+
+process.removeAllListeners('warning')
+
+test('cannot remove all content type parsers after binding', async (t) => {
+  t.plan(1)
+
+  const fastify = Fastify()
+
+  t.after(() => fastify.close())
+
+  await fastify.listen({ port: 0 })
+  t.assert.throws(() => fastify.removeAllContentTypeParsers())
+})
+
+test('cannot remove content type parsers after binding', async (t) => {
+  t.plan(1)
+
+  const fastify = Fastify()
+  t.after(() => fastify.close())
+
+  await fastify.listen({ port: 0 })
+  t.assert.throws(() => fastify.removeContentTypeParser('application/json'))
+})
+
+test('should be able to override the default json parser after removeAllContentTypeParsers', async (t) => {
+  t.plan(4)
+
+  const fastify = Fastify()
+
+  fastify.post('/', (req, reply) => {
+    reply.send(req.body)
+  })
+
+  fastify.removeAllContentTypeParsers()
+
+  fastify.addContentTypeParser('application/json', function (req, payload, done) {
+    t.assert.ok('called')
+    jsonParser(payload, function (err, body) {
+      done(err, body)
+    })
+  })
+
+  const fastifyServer = await fastify.listen({ port: 0 })
+
+  const result = await fetch(fastifyServer, {
+    method: 'POST',
+    body: JSON.stringify({ hello: 'world' }),
+    headers: {
+      'Content-Type': 'application/json'
+    }
+  })
+
+  t.assert.ok(result.ok)
+  t.assert.strictEqual(result.status, 200)
+  t.assert.deepStrictEqual(await result.text(), JSON.stringify({ hello: 'world' }))
+  await fastify.close()
+})
+
+test('should be able to override the default plain text parser after removeAllContentTypeParsers', async (t) => {
+  t.plan(4)
+
+  const fastify = Fastify()
+
+  fastify.post('/', (req, reply) => {
+    reply.send(req.body)
+  })
+
+  fastify.removeAllContentTypeParsers()
+
+  fastify.addContentTypeParser('text/plain', function (req, payload, done) {
+    t.assert.ok('called')
+    plainTextParser(payload, function (err, body) {
+      done(err, body)
+    })
+  })
+
+  const fastifyServer = await fastify.listen({ port: 0 })
+
+  const result = await fetch(fastifyServer, {
+    method: 'POST',
+    body: 'hello world',
+    headers: {
+      'Content-Type': 'text/plain'
+    }
+  })
+
+  t.assert.ok(result.ok)
+  t.assert.strictEqual(result.status, 200)
+  t.assert.strictEqual(await result.text(), 'hello world')
+  await fastify.close()
+})
+
+test('should be able to add a custom content type parser after removeAllContentTypeParsers', async (t) => {
+  t.plan(4)
+
+  const fastify = Fastify()
+
+  fastify.post('/', (req, reply) => {
+    reply.send(req.body)
+  })
+
+  fastify.removeAllContentTypeParsers()
+  fastify.addContentTypeParser('application/jsoff', function (req, payload, done) {
+    t.assert.ok('called')
+    jsonParser(payload, function (err, body) {
+      done(err, body)
+    })
+  })
+
+  const fastifyServer = await fastify.listen({ port: 0 })
+
+  const result = await fetch(fastifyServer, {
+    method: 'POST',
+    body: JSON.stringify({ hello: 'world' }),
+    headers: {
+      'Content-Type': 'application/jsoff'
+    }
+  })
+
+  t.assert.ok(result.ok)
+  t.assert.strictEqual(result.status, 200)
+  t.assert.deepStrictEqual(await result.text(), JSON.stringify({ hello: 'world' }))
+  await fastify.close()
+})

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

@@ -0,0 +1,129 @@
+'use strict'
+
+const { test } = require('node:test')
+const querystring = require('node:querystring')
+const Fastify = require('..')
+
+test('Custom querystring parser', async t => {
+  t.plan(7)
+
+  const fastify = Fastify({
+    querystringParser: function (str) {
+      t.assert.strictEqual(str, 'foo=bar&baz=faz')
+      return querystring.parse(str)
+    }
+  })
+
+  fastify.get('/', (req, reply) => {
+    t.assert.deepEqual(req.query, {
+      foo: 'bar',
+      baz: 'faz'
+    })
+    reply.send({ hello: 'world' })
+  })
+
+  const fastifyServer = await fastify.listen({ port: 0 })
+  t.after(() => fastify.close())
+
+  const result = await fetch(`${fastifyServer}?foo=bar&baz=faz`)
+  t.assert.ok(result.ok)
+  t.assert.strictEqual(result.status, 200)
+
+  const injectResponse = await fastify.inject({
+    method: 'GET',
+    url: `${fastifyServer}?foo=bar&baz=faz`
+  })
+  t.assert.strictEqual(injectResponse.statusCode, 200)
+})
+
+test('Custom querystring parser should be called also if there is nothing to parse', async t => {
+  t.plan(7)
+
+  const fastify = Fastify({
+    querystringParser: function (str) {
+      t.assert.strictEqual(str, '')
+      return querystring.parse(str)
+    }
+  })
+
+  fastify.get('/', (req, reply) => {
+    t.assert.deepEqual(req.query, {})
+    reply.send({ hello: 'world' })
+  })
+
+  const fastifyServer = await fastify.listen({ port: 0 })
+  t.after(() => fastify.close())
+
+  const result = await fetch(fastifyServer)
+  t.assert.ok(result.ok)
+  t.assert.strictEqual(result.status, 200)
+
+  const injectResponse = await fastify.inject({
+    method: 'GET',
+    url: fastifyServer
+  })
+  t.assert.strictEqual(injectResponse.statusCode, 200)
+})
+
+test('Querystring without value', async t => {
+  t.plan(7)
+
+  const fastify = Fastify({
+    querystringParser: function (str) {
+      t.assert.strictEqual(str, 'foo')
+      return querystring.parse(str)
+    }
+  })
+
+  fastify.get('/', (req, reply) => {
+    t.assert.deepEqual(req.query, { foo: '' })
+    reply.send({ hello: 'world' })
+  })
+
+  const fastifyServer = await fastify.listen({ port: 0 })
+  t.after(() => fastify.close())
+
+  const result = await fetch(`${fastifyServer}?foo`)
+  t.assert.ok(result.ok)
+  t.assert.strictEqual(result.status, 200)
+
+  const injectResponse = await fastify.inject({
+    method: 'GET',
+    url: `${fastifyServer}?foo`
+  })
+  t.assert.strictEqual(injectResponse.statusCode, 200)
+})
+
+test('Custom querystring parser should be a function', t => {
+  t.plan(1)
+
+  try {
+    Fastify({
+      querystringParser: 10
+    })
+    t.assert.fail('Should throw')
+  } catch (err) {
+    t.assert.strictEqual(
+      err.message,
+      "querystringParser option should be a function, instead got 'number'"
+    )
+  }
+})
+
+test('Custom querystring parser should be a function', t => {
+  t.plan(1)
+
+  try {
+    Fastify({
+      routerOptions: {
+        querystringParser: 10
+      }
+    })
+    t.fail('Should throw')
+  } catch (err) {
+    t.assert.equal(
+      err.message,
+      "querystringParser option should be a function, instead got 'number'"
+    )
+  }
+})