-
-
Notifications
You must be signed in to change notification settings - Fork 287
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
deps: update fastify to 4.19.0 #5726
Conversation
export class ApiError extends Error { | ||
statusCode: number; | ||
constructor(statusCode: number, message?: string) { | ||
statusCode: HttpErrorCodes; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is required due to fastify/fastify#4823, if number
type is used only a undefined
payload can be passed to send()
. Anyhow a good idea to narrow this type here down to only http error codes.
// eslint-disable-next-line import/no-extraneous-dependencies | ||
import type * as fastify from "fastify"; | ||
import {ReqGeneric} from "../types.js"; | ||
|
||
export type ServerInstance = FastifyInstance; | ||
|
||
export type RouteConfig = { | ||
export type RouteConfig = FastifyContextConfig & { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Required due to
Performance Report✔️ no performance regression detected Full benchmark results
|
🎉 This PR is included in v1.10.0 🎉 |
Motivation
Keep dependencies up to date
Description
This update includes better error response if URI is too long which we had multiple times in the past