fastify 4.25.0 → 4.25.1

package/README.md

@@ -377,10 +377,10 @@ to join this group by Lead Maintainers.
 
 [<img
 src="https://github.com/openjs-foundation/artwork/blob/main/openjs_foundation/openjs_foundation-logo-horizontal-color.png?raw=true"
-width="250px;"/>](https://openjsf.org/projects/#growth)
+width="250px;"/>](https://openjsf.org/projects)
 
-We are a [Growth
-Project](https://github.com/openjs-foundation/cross-project-council/blob/HEAD/PROJECT_PROGRESSION.md#growth-stage)
+We are a [At-Large
+Project](https://github.com/openjs-foundation/cross-project-council/blob/HEAD/PROJECT_PROGRESSION.md#at-large-projects)
 in the [OpenJS Foundation](https://openjsf.org/).
 
 ## Acknowledgements

package/fastify.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const VERSION = '4.25.0'
+const VERSION = '4.25.1'
 
 const Avvio = require('avvio')
 const http = require('node:http')

package/lib/pluginUtils.js

@@ -136,14 +136,14 @@ function registerPluginName (fn) {
   return name
 }
 
-function checkPluginHealthiness (fn, pluginName = 'anonymous') {
+function checkPluginHealthiness (fn, pluginName) {
   if (fn.constructor.name === 'AsyncFunction' && fn.length === 3) {
-    FSTWRN002(pluginName)
+    FSTWRN002(pluginName || 'anonymous')
   }
 }
 
 function registerPlugin (fn) {
-  const pluginName = registerPluginName.call(this, fn)
+  const pluginName = registerPluginName.call(this, fn) || getPluginName(fn)
   checkPluginHealthiness.call(this, fn, pluginName)
   checkVersion.call(this, fn)
   checkDecorators.call(this, fn)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fastify",
-  "version": "4.25.0",
+  "version": "4.25.1",
   "description": "Fast and low overhead web framework, for Node.js",
   "main": "fastify.js",
   "type": "commonjs",

package/test/plugin.4.test.js

@@ -416,20 +416,27 @@ test('hasPlugin returns true when using encapsulation', async t => {
 })
 
 test('registering plugins with mixed style should return a warning', async t => {
-  t.plan(4)
+  t.plan(12)
+
+  const pluginNames = ['error-plugin', 'anonymous', 'anotherPlugin', 'anotherPluginNamed']
 
+  const oldWarnings = process.listeners('warning')
+  process.removeAllListeners('warning')
   process.on('warning', onWarning)
   function onWarning (warning) {
+    t.match(warning.message, new RegExp(`.*${pluginNames.shift()} plugin being registered mixes async and callback styles.*`))
     t.equal(warning.name, 'FastifyWarning')
     t.equal(warning.code, 'FSTWRN002')
   }
+  t.teardown(() => {
+    process.removeListener('warning', onWarning)
+    for (const warning of oldWarnings) {
+      process.on('warning', warning)
+    }
+  })
 
   const fastify = Fastify()
 
-  const anonymousPlugin = async (app, opts, done) => {
-    done()
-  }
-
   const pluginName = 'error-plugin'
   const errorPlugin = async (app, opts, done) => {
     done()
@@ -437,8 +444,20 @@ test('registering plugins with mixed style should return a warning', async t =>
 
   const namedPlugin = fp(errorPlugin, { name: pluginName })
 
+  async function anotherPlugin (app, opts, done) {
+    done()
+  }
+
+  const anotherPluginNamed = async function (app, opts, done) {
+    done()
+  }
+
   fastify.register(namedPlugin)
-  fastify.register(anonymousPlugin)
+  fastify.register(async (app, opts, done) => {
+    done()
+  })
+  fastify.register(anotherPlugin)
+  fastify.register(anotherPluginNamed)
 
   await fastify.ready()
 })

package/test/types/route.test-d.ts

@@ -444,19 +444,12 @@ expectType<boolean>(fastify().hasRoute({
   url: '/',
   method: 'GET',
   constraints: {
-    something: {
-      name: 'secret',
-      storage: function () {
-        return {
-          get: (type) => {
-            return null
-          },
-          set: (type, store) => {}
-        }
-      },
-      deriveConstraint: (req, ctx, done) => {},
-      mustMatchWhenDerived: true
-    }
+    // constraints value should accept any value
+    number: 12,
+    date: new Date(),
+    boolean: true,
+    function: () => {},
+    object: { foo: 'bar' }
   }
 }))
 

package/types/route.d.ts

@@ -1,4 +1,5 @@
 import { FastifyError } from '@fastify/error'
+import { ConstraintStrategy } from 'find-my-way'
 import { FastifyRequestContext } from './context'
 import { onErrorMetaHookHandler, onRequestAbortMetaHookHandler, onRequestMetaHookHandler, onResponseMetaHookHandler, onSendMetaHookHandler, onTimeoutMetaHookHandler, preHandlerMetaHookHandler, preParsingMetaHookHandler, preSerializationMetaHookHandler, preValidationMetaHookHandler } from './hooks'
 import { FastifyInstance } from './instance'
@@ -12,7 +13,6 @@ import {
   ResolveFastifyReplyReturnType
 } from './type-provider'
 import { ContextConfigDefault, HTTPMethods, RawReplyDefaultExpression, RawRequestDefaultExpression, RawServerBase, RawServerDefault } from './utils'
-import { ConstraintStrategy } from 'find-my-way'
 
 export interface FastifyRouteConfig {
   url: string;
@@ -25,6 +25,12 @@ export type RouteConstraintType = Omit<ConstraintStrategy<any>, 'deriveConstrain
   deriveConstraint<Context>(req: RawRequestDefaultExpression<RawServerDefault>, ctx?: Context, done?: (err: Error, ...args: any) => any): any,
 }
 
+export interface RouteConstraint {
+  version?: string
+  host?: RegExp | string
+  [name: string]: unknown
+}
+
 /**
  * Route shorthand options for the various shorthand methods
  */
@@ -48,7 +54,7 @@ export interface RouteShorthandOptions<
   logLevel?: LogLevel;
   config?: Omit<FastifyRequestContext<ContextConfig>['config'], 'url' | 'method'>;
   version?: string;
-  constraints?: { version: string } | {host: RegExp | string} | {[name: string]: RouteConstraintType },
+  constraints?: RouteConstraint,
   prefixTrailingSlash?: 'slash'|'no-slash'|'both';
   errorHandler?: (
     this: FastifyInstance<RawServer, RawRequest, RawReply, Logger, TypeProvider>,