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
Copy link
Copy Markdown

@daimalou daimalou commented Jan 21, 2025

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
Copy link
Copy Markdown

@themavik themavik left a comment

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)—rename 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