Skip to content

feat: trace pino transports in constructor and fastify patterns#573

Merged
jeffsee55 merged 1 commit intomainfrom
jeffsee55/pino-constructor-fastify
Mar 20, 2026
Merged

feat: trace pino transports in constructor and fastify patterns#573
jeffsee55 merged 1 commit intomainfrom
jeffsee55/pino-constructor-fastify

Conversation

@jeffsee55
Copy link
Contributor

The previous pino support only handled pino.transport({ target: '...' }). This adds support for two additional patterns:

  • pino({ transport: { target: '...' } }) (constructor config)
  • fastify({ logger: { transport: { target: '...' } } }) (fastify logger)

Also extracts shared transport config processing into a reusable helper.

@jeffsee55 jeffsee55 requested review from a team, icyJoseph, ijjk and styfle as code owners March 20, 2026 19:20
The previous pino support only handled `pino.transport({ target: '...' })`.
This adds support for two additional patterns:
- `pino({ transport: { target: '...' } })` (constructor config)
- `fastify({ logger: { transport: { target: '...' } } })` (fastify logger)

Also extracts shared transport config processing into a reusable helper.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jeffsee55 jeffsee55 force-pushed the jeffsee55/pino-constructor-fastify branch from b20ffb5 to 958f4d4 Compare March 20, 2026 20:03
@jeffsee55 jeffsee55 requested a review from styfle March 20, 2026 20:18
@jeffsee55 jeffsee55 merged commit a044730 into main Mar 20, 2026
12 checks passed
@jeffsee55 jeffsee55 deleted the jeffsee55/pino-constructor-fastify branch March 20, 2026 21:00
@github-actions
Copy link

🎉 This PR is included in version 1.5.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants