Skip to content

feat(platform-fastify): add fastify custom log level feature - #14470

Open
daimalou wants to merge 1 commit into
nestjs:masterfrom
daimalou:master
Open

feat(platform-fastify): add fastify custom log level feature#14470
daimalou wants to merge 1 commit into
nestjs:masterfrom
daimalou:master

Conversation

@daimalou

@daimalou daimalou commented Jan 21, 2025

Copy link
Copy Markdown

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Other... Please describe:

What is the current behavior?

can not close fastify buildin log

Issue Number: N/A
#14137

use fastify 5

https://fastify.dev/docs/latest/Reference/Routes/#custom-log-level

What is the new behavior?

close fastify buildin log

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@daimalou daimalou changed the title add fastify custom log level feature feat(platform-fastify): add fastify custom log level feature Jan 21, 2025

@themavik themavik left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FastifyAdapter wiring for FASTIFY_ROUTE_LOG_METADATA and spreading routeLog into route options matches Fastify per-route log config. nit: router-log.decorator.spec.ts still uses describe(@RouteConfig)鈥攔ename so failures point at @RouteLog.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants