fastify 5.0.0-alpha.2 → 5.0.0-alpha.4

package/lib/warnings.js

@@ -1,91 +1,13 @@
 'use strict'
 
-const { createDeprecation, createWarning } = require('process-warning')
+const { createWarning } = require('process-warning')
 
 /**
  * Deprecation codes:
- *   - FSTDEP005
- *   - FSTDEP007
- *   - FSTDEP008
- *   - FSTDEP009
- *   - FSTDEP010
- *   - FSTDEP012
- *   - FSTDEP013
- *   - FSTDEP015
- *   - FSTDEP016
- *   - FSTDEP017
- *   - FSTDEP018
- *   - FSTDEP019
- *   - FSTDEP021
  *   - FSTWRN001
  *   - FSTSEC001
  */
 
-const FSTDEP005 = createDeprecation({
-  code: 'FSTDEP005',
-  message: 'You are accessing the deprecated "request.connection" property. Use "request.socket" instead.'
-})
-
-const FSTDEP007 = createDeprecation({
-  code: 'FSTDEP007',
-  message: 'You are trying to set a HEAD route using "exposeHeadRoute" route flag when a sibling route is already set. See documentation for more info.'
-})
-
-const FSTDEP008 = createDeprecation({
-  code: 'FSTDEP008',
-  message: 'You are using route constraints via the route { version: "..." } option, use { constraints: { version: "..." } } option instead.'
-})
-
-const FSTDEP009 = createDeprecation({
-  code: 'FSTDEP009',
-  message: 'You are using a custom route versioning strategy via the server { versioning: "..." } option, use { constraints: { version: "..." } } option instead.'
-})
-
-const FSTDEP010 = createDeprecation({
-  code: 'FSTDEP010',
-  message: 'Modifying the "reply.sent" property is deprecated. Use the "reply.hijack()" method instead.'
-})
-
-const FSTDEP012 = createDeprecation({
-  code: 'FSTDEP012',
-  message: 'request.context property access is deprecated. Please use "request.routeOptions.config" or "request.routeOptions.schema" instead for accessing Route settings. The "request.context" will be removed in `fastify@5`.'
-})
-
-const FSTDEP013 = createDeprecation({
-  code: 'FSTDEP013',
-  message: 'Direct return of "trailers" function is deprecated. Please use "callback" or "async-await" for return value. The support of direct return will removed in `fastify@5`.'
-})
-
-const FSTDEP015 = createDeprecation({
-  code: 'FSTDEP015',
-  message: 'You are accessing the deprecated "request.routeSchema" property. Use "request.routeOptions.schema" instead. Property "req.routeSchema" will be removed in `fastify@5`.'
-})
-
-const FSTDEP016 = createDeprecation({
-  code: 'FSTDEP016',
-  message: 'You are accessing the deprecated "request.routeConfig" property. Use "request.routeOptions.config" instead. Property "req.routeConfig" will be removed in `fastify@5`.'
-})
-
-const FSTDEP017 = createDeprecation({
-  code: 'FSTDEP017',
-  message: 'You are accessing the deprecated "request.routerPath" property. Use "request.routeOptions.url" instead. Property "req.routerPath" will be removed in `fastify@5`.'
-})
-
-const FSTDEP018 = createDeprecation({
-  code: 'FSTDEP018',
-  message: 'You are accessing the deprecated "request.routerMethod" property. Use "request.routeOptions.method" instead. Property "req.routerMethod" will be removed in `fastify@5`.'
-})
-
-const FSTDEP019 = createDeprecation({
-  code: 'FSTDEP019',
-  message: 'reply.context property access is deprecated. Please use "request.routeOptions.config" or "request.routeOptions.schema" instead for accessing Route settings. The "reply.context" will be removed in `fastify@5`.'
-})
-
-const FSTDEP021 = createDeprecation({
-  code: 'FSTDEP021',
-  message: 'The `reply.redirect()` method has a new signature: `reply.redirect(url: string, code?: number)`. It will be enforced in `fastify@v5`'
-})
-
 const FSTWRN001 = createWarning({
   name: 'FastifyWarning',
   code: 'FSTWRN001',
@@ -101,19 +23,6 @@ const FSTSEC001 = createWarning({
 })
 
 module.exports = {
-  FSTDEP005,
-  FSTDEP007,
-  FSTDEP008,
-  FSTDEP009,
-  FSTDEP010,
-  FSTDEP012,
-  FSTDEP013,
-  FSTDEP015,
-  FSTDEP016,
-  FSTDEP017,
-  FSTDEP018,
-  FSTDEP019,
-  FSTDEP021,
   FSTWRN001,
   FSTSEC001
 }

package/lib/wrapThenable.js

@@ -5,7 +5,7 @@ const {
   kReplyHijacked
 } = require('./symbols')
 
-const diagnostics = require('dc-polyfill')
+const diagnostics = require('node:diagnostics_channel')
 const channels = diagnostics.tracingChannel('fastify.request.handler')
 
 function wrapThenable (thenable, reply, store) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fastify",
-  "version": "5.0.0-alpha.2",
+  "version": "5.0.0-alpha.4",
   "description": "Fast and low overhead web framework, for Node.js",
   "main": "fastify.js",
   "type": "commonjs",
@@ -22,7 +22,7 @@
     "test:ci": "npm run unit -- --coverage-report=lcovonly && npm run test:typescript",
     "test:report": "npm run lint && npm run unit:report && npm run test:typescript",
     "test:validator:integrity": "npm run build:validation && git diff --quiet --ignore-all-space --ignore-blank-lines --ignore-cr-at-eol lib/error-serializer.js && git diff --quiet --ignore-all-space --ignore-blank-lines --ignore-cr-at-eol lib/configValidator.js",
-    "test:typescript": "tsc test/types/import.ts && tsd",
+    "test:typescript": "tsc test/types/import.ts --noEmit && tsd",
     "test:watch": "npm run unit -- --watch --coverage-report=none --reporter=terse",
     "unit": "tap",
     "unit:junit": "tap-mocha-reporter xunit < out.tap > test/junit-testresults.xml",
@@ -152,11 +152,11 @@
   ],
   "devDependencies": {
     "@fastify/pre-commit": "^2.1.0",
-    "@sinclair/typebox": "^0.32.22",
+    "@sinclair/typebox": "^0.33.4",
     "@sinonjs/fake-timers": "^11.2.2",
     "@stylistic/eslint-plugin": "^2.1.0",
     "@stylistic/eslint-plugin-js": "^2.1.0",
-    "@types/node": "^20.12.7",
+    "@types/node": "^22.0.0",
     "ajv": "^8.12.0",
     "ajv-errors": "^3.0.0",
     "ajv-formats": "^3.0.1",
@@ -169,20 +169,20 @@
     "eslint": "^9.0.0",
     "fast-json-body": "^1.1.0",
     "fastify-plugin": "^4.5.1",
-    "fluent-json-schema": "^4.2.1",
+    "fluent-json-schema": "^5.0.0",
     "h2url": "^0.2.0",
     "http-errors": "^2.0.0",
     "joi": "^17.12.3",
     "json-schema-to-ts": "^3.0.1",
     "JSONStream": "^1.3.5",
     "markdownlint-cli2": "^0.13.0",
-    "neostandard": "^0.7.0",
+    "neostandard": "^0.11.3",
     "node-forge": "^1.3.1",
     "proxyquire": "^2.1.3",
     "send": "^0.18.0",
     "simple-get": "^4.0.1",
     "split2": "^4.2.0",
-    "tap": "^19.0.0",
+    "tap": "^21.0.0",
     "tsd": "^0.31.0",
     "typescript": "^5.4.5",
     "undici": "^6.13.0",
@@ -190,17 +190,16 @@
     "yup": "^1.4.0"
   },
   "dependencies": {
-    "@fastify/ajv-compiler": "^3.5.0",
-    "@fastify/error": "^3.4.1",
+    "@fastify/ajv-compiler": "^4.0.0",
+    "@fastify/error": "^4.0.0",
     "@fastify/fast-json-stringify-compiler": "^4.3.0",
     "abstract-logging": "^2.0.1",
-    "avvio": "^8.3.0",
-    "dc-polyfill": "^0.1.6",
-    "fast-json-stringify": "^5.14.1",
-    "find-my-way": "^8.1.0",
-    "light-my-request": "^5.13.0",
+    "avvio": "^9.0.0",
+    "fast-json-stringify": "^6.0.0",
+    "find-my-way": "^9.0.0",
+    "light-my-request": "^6.0.0",
     "pino": "^9.0.0",
-    "process-warning": "^3.0.0",
+    "process-warning": "^4.0.0",
     "proxy-addr": "^2.0.7",
     "rfdc": "^1.3.1",
     "secure-json-parse": "^2.7.0",

package/test/decorator.test.js

@@ -639,7 +639,7 @@ test('should register empty values', t => {
 
   fastify.register((instance, opts, done) => {
     instance.decorate('test', null)
-    t.ok(Object.prototype.hasOwnProperty.call(instance, 'test'))
+    t.ok(Object.hasOwn(instance, 'test'))
     done()
   })
 

package/test/diagnostics-channel/404.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const t = require('tap')
-const diagnostics = require('dc-polyfill')
+const diagnostics = require('node:diagnostics_channel')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('../..')

package/test/diagnostics-channel/async-delay-request.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const t = require('tap')
-const diagnostics = require('dc-polyfill')
+const diagnostics = require('node:diagnostics_channel')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('../..')

package/test/diagnostics-channel/async-request.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const t = require('tap')
-const diagnostics = require('dc-polyfill')
+const diagnostics = require('node:diagnostics_channel')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('../..')

package/test/diagnostics-channel/error-before-handler.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const t = require('tap')
-const diagnostics = require('dc-polyfill')
+const diagnostics = require('node:diagnostics_channel')
 const test = t.test
 require('../../lib/hooks').onSendHookRunner = function Stub () {}
 const Request = require('../../lib/request')

package/test/diagnostics-channel/error-request.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const t = require('tap')
-const diagnostics = require('dc-polyfill')
+const diagnostics = require('node:diagnostics_channel')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('../..')

package/test/diagnostics-channel/error-status.test.js

@@ -4,7 +4,7 @@ const t = require('tap')
 const test = t.test
 const Fastify = require('../..')
 const statusCodes = require('node:http').STATUS_CODES
-const diagnostics = require('dc-polyfill')
+const diagnostics = require('node:diagnostics_channel')
 
 test('Error.status property support', t => {
   t.plan(4)

package/test/diagnostics-channel/init.test.js

@@ -24,7 +24,7 @@ test('diagnostics_channel when present and subscribers', t => {
   }
 
   const fastify = proxyquire('../../fastify', {
-    'dc-polyfill': diagnostics
+    'node:diagnostics_channel': diagnostics
   })()
   t.equal(fastifyInHook, fastify)
 })
@@ -46,6 +46,6 @@ test('diagnostics_channel when present and no subscribers', t => {
   }
 
   proxyquire('../../fastify', {
-    'dc-polyfill': diagnostics
+    'node:diagnostics_channel': diagnostics
   })()
 })

package/test/diagnostics-channel/sync-delay-request.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const t = require('tap')
-const diagnostics = require('dc-polyfill')
+const diagnostics = require('node:diagnostics_channel')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('../..')

package/test/diagnostics-channel/sync-request-reply.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const t = require('tap')
-const diagnostics = require('dc-polyfill')
+const diagnostics = require('node:diagnostics_channel')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('../..')

package/test/diagnostics-channel/sync-request.test.js

@@ -1,7 +1,7 @@
 'use strict'
 
 const t = require('tap')
-const diagnostics = require('dc-polyfill')
+const diagnostics = require('node:diagnostics_channel')
 const test = t.test
 const sget = require('simple-get').concat
 const Fastify = require('../..')

package/test/{copy.test.js → http-methods/copy.test.js}

@@ -3,7 +3,8 @@
 const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
-const fastify = require('..')()
+const fastify = require('../../fastify')()
+fastify.addHttpMethod('COPY')
 
 test('can be created - copy', t => {
   t.plan(1)

package/test/http-methods/custom-http-methods.test.js

@@ -0,0 +1,111 @@
+'use strict'
+
+const http = require('node:http')
+const { test } = require('tap')
+const Fastify = require('../../fastify')
+
+function addEcho (fastify, method) {
+  fastify.route({
+    method,
+    url: '/',
+    handler: function (req, reply) {
+      reply.send(req.body)
+    }
+  })
+}
+
+test('missing method from http client', t => {
+  t.plan(2)
+  const fastify = Fastify()
+
+  fastify.listen({ port: 3000 }, (err) => {
+    t.error(err)
+
+    const port = fastify.server.address().port
+    const req = http.request({
+      port,
+      method: 'REBIND',
+      path: '/'
+    }, (res) => {
+      t.equal(res.statusCode, 404)
+      fastify.close()
+    })
+
+    req.end()
+  })
+})
+
+test('addHttpMethod increase the supported HTTP methods supported', t => {
+  t.plan(8)
+  const app = Fastify()
+
+  t.throws(() => { addEcho(app, 'REBIND') }, /REBIND method is not supported./)
+  t.notOk(app.supportedMethods.includes('REBIND'))
+  t.notOk(app.rebind)
+
+  app.addHttpMethod('REBIND')
+  t.doesNotThrow(() => { addEcho(app, 'REBIND') }, 'REBIND method is supported.')
+  t.ok(app.supportedMethods.includes('REBIND'))
+  t.ok(app.rebind)
+
+  app.rebind('/foo', () => 'hello')
+
+  app.inject({
+    method: 'REBIND',
+    url: '/foo'
+  }, (err, response) => {
+    t.error(err)
+    t.equal(response.payload, 'hello')
+  })
+})
+
+test('addHttpMethod adds a new custom method without body', t => {
+  t.plan(3)
+  const app = Fastify()
+
+  t.throws(() => { addEcho(app, 'REBIND') }, /REBIND method is not supported./)
+
+  app.addHttpMethod('REBIND')
+  t.doesNotThrow(() => { addEcho(app, 'REBIND') }, 'REBIND method is supported.')
+
+  t.throws(() => {
+    app.route({
+      url: '/',
+      method: 'REBIND',
+      schema: {
+        body: {
+          type: 'object',
+          properties: {
+            hello: { type: 'string' }
+          }
+        }
+      },
+      handler: function (req, reply) {
+        reply.send(req.body)
+      }
+    })
+  }, /Body validation schema for REBIND:\/ route is not supported!/)
+})
+
+test('addHttpMethod adds a new custom method with body', t => {
+  t.plan(3)
+  const app = Fastify()
+
+  app.addHttpMethod('REBIND', { hasBody: true })
+  t.doesNotThrow(() => { addEcho(app, 'REBIND') }, 'REBIND method is supported.')
+
+  app.inject({
+    method: 'REBIND',
+    url: '/',
+    payload: { hello: 'world' }
+  }, (err, response) => {
+    t.error(err)
+    t.same(response.json(), { hello: 'world' })
+  })
+})
+
+test('addHttpMethod rejects fake http method', t => {
+  t.plan(1)
+  const fastify = Fastify()
+  t.throws(() => { fastify.addHttpMethod('FOOO') }, /Provided method is invalid!/)
+})

package/test/{get.test.js → http-methods/get.test.js}

@@ -3,7 +3,7 @@
 const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
-const fastify = require('..')()
+const fastify = require('../../fastify')()
 
 const schema = {
   schema: {

package/test/{head.test.js → http-methods/head.test.js}

@@ -3,7 +3,7 @@
 const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
-const fastify = require('..')()
+const fastify = require('../../fastify')()
 
 const schema = {
   schema: {
@@ -59,12 +59,12 @@ test('shorthand - head', t => {
 test('shorthand - custom head', t => {
   t.plan(1)
   try {
-    fastify.get('/proxy/*', function (req, reply) {
+    fastify.head('/proxy/*', function (req, reply) {
+      reply.headers({ 'x-foo': 'bar' })
       reply.code(200).send(null)
     })
 
-    fastify.head('/proxy/*', function (req, reply) {
-      reply.headers({ 'x-foo': 'bar' })
+    fastify.get('/proxy/*', function (req, reply) {
       reply.code(200).send(null)
     })
 
@@ -77,12 +77,12 @@ test('shorthand - custom head', t => {
 test('shorthand - custom head with constraints', t => {
   t.plan(1)
   try {
-    fastify.get('/proxy/*', { constraints: { version: '1.0.0' } }, function (req, reply) {
+    fastify.head('/proxy/*', { constraints: { version: '1.0.0' } }, function (req, reply) {
+      reply.headers({ 'x-foo': 'bar' })
       reply.code(200).send(null)
     })
 
-    fastify.head('/proxy/*', { constraints: { version: '1.0.0' } }, function (req, reply) {
-      reply.headers({ 'x-foo': 'bar' })
+    fastify.get('/proxy/*', { constraints: { version: '1.0.0' } }, function (req, reply) {
       reply.code(200).send(null)
     })
 
@@ -109,62 +109,6 @@ test('shorthand - should not reset a head route', t => {
   }
 })
 
-test('shorthand - should override head route when setting multiple routes', t => {
-  t.plan(1)
-  try {
-    fastify.route({
-      method: 'GET',
-      url: '/query2',
-      handler: function (req, reply) {
-        reply.headers({ 'x-foo': 'bar' })
-        reply.code(200).send(null)
-      }
-    })
-
-    fastify.route({
-      method: ['POST', 'PUT', 'HEAD'],
-      url: '/query2',
-      handler: function (req, reply) {
-        reply.headers({ 'x-foo': 'bar' })
-        reply.code(200).send(null)
-      }
-    })
-
-    t.pass()
-  } catch (e) {
-    console.log(e)
-    t.fail()
-  }
-})
-
-test('shorthand - should override head route when setting multiple routes', t => {
-  t.plan(1)
-  try {
-    fastify.route({
-      method: ['GET'],
-      url: '/query3',
-      handler: function (req, reply) {
-        reply.headers({ 'x-foo': 'bar' })
-        reply.code(200).send(null)
-      }
-    })
-
-    fastify.route({
-      method: ['POST', 'PUT', 'HEAD'],
-      url: '/query3',
-      handler: function (req, reply) {
-        reply.headers({ 'x-foo': 'bar' })
-        reply.code(200).send(null)
-      }
-    })
-
-    t.pass()
-  } catch (e) {
-    console.log(e)
-    t.fail()
-  }
-})
-
 test('shorthand - should set get and head route in the same api call', t => {
   t.plan(1)
   try {
@@ -330,30 +274,6 @@ fastify.listen({ port: 0 }, err => {
     })
   })
 
-  test('shorthand - should override head route when setting multiple routes', t => {
-    t.plan(3)
-    sget({
-      method: 'HEAD',
-      url: 'http://localhost:' + fastify.server.address().port + '/query2'
-    }, (err, response) => {
-      t.error(err)
-      t.equal(response.headers['x-foo'], 'bar')
-      t.equal(response.statusCode, 200)
-    })
-  })
-
-  test('shorthand - should override head route when setting multiple routes', t => {
-    t.plan(3)
-    sget({
-      method: 'HEAD',
-      url: 'http://localhost:' + fastify.server.address().port + '/query3'
-    }, (err, response) => {
-      t.error(err)
-      t.equal(response.headers['x-foo'], 'bar')
-      t.equal(response.statusCode, 200)
-    })
-  })
-
   test('shorthand - should set get and head route in the same api call', t => {
     t.plan(3)
     sget({

package/test/{lock.test.js → http-methods/lock.test.js}

@@ -3,7 +3,8 @@
 const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
-const fastify = require('..')()
+const fastify = require('../../fastify')()
+fastify.addHttpMethod('LOCK', { hasBody: true })
 
 const bodySample = `<?xml version="1.0" encoding="utf-8" ?>
         <D:lockinfo xmlns:D='DAV:'>

package/test/{mkcalendar.test.js → http-methods/mkcalendar.test.js}

@@ -3,7 +3,8 @@
 const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
-const fastify = require('../fastify')()
+const fastify = require('../../fastify')()
+fastify.addHttpMethod('MKCALENDAR', { hasBody: true })
 
 const bodySample = `<?xml version="1.0" encoding="UTF-8"?>
   <B:mkcalendar xmlns:B="urn:ietf:params:xml:ns:caldav">

package/test/{mkcol.test.js → http-methods/mkcol.test.js}

@@ -3,7 +3,8 @@
 const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
-const fastify = require('..')()
+const fastify = require('../../')()
+fastify.addHttpMethod('MKCOL')
 
 test('can be created - mkcol', t => {
   t.plan(1)

package/test/{move.test.js → http-methods/move.test.js}

@@ -3,7 +3,8 @@
 const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
-const fastify = require('..')()
+const fastify = require('../../')()
+fastify.addHttpMethod('MOVE')
 
 test('shorthand - move', t => {
   t.plan(1)

package/test/{propfind.test.js → http-methods/propfind.test.js}

@@ -3,7 +3,8 @@
 const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
-const fastify = require('..')()
+const fastify = require('../../')()
+fastify.addHttpMethod('PROPFIND', { hasBody: true })
 
 const bodySample = `<?xml version="1.0" encoding="utf-8" ?>
         <D:propfind xmlns:D="DAV:">

package/test/{proppatch.test.js → http-methods/proppatch.test.js}

@@ -3,7 +3,8 @@
 const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
-const fastify = require('..')()
+const fastify = require('../../')()
+fastify.addHttpMethod('PROPPATCH', { hasBody: true })
 
 const bodySample = `<?xml version="1.0" encoding="utf-8" ?>
         <D:propertyupdate xmlns:D="DAV:"

package/test/{report.test.js → http-methods/report.test.js}

@@ -3,7 +3,8 @@
 const t = require('tap')
 const test = t.test
 const sget = require('simple-get').concat
-const fastify = require('../fastify')()
+const fastify = require('../../fastify')()
+fastify.addHttpMethod('REPORT', { hasBody: true })
 
 const bodySample = `<?xml version="1.0" encoding="UTF-8"?>
   <B:calendar-query xmlns:B="urn:ietf:params:xml:ns:caldav">