feat: send config to transports on init by 10xLaCroixDrinker · Pull Request #1930 · pinojs/pino

10xLaCroixDrinker · 2024-03-22T22:06:59Z

PR 2 of 3 for #1925

Relates to pinojs/thread-stream#145
Relates to pinojs/pino-abstract-transport#90

Include pinoWillSendConfig in workerData so that transports using the config know to wait, and emit a message event with PINO_CONFIG on the stream when initializing Pino

mcollina · 2024-04-22T12:33:33Z

@10xLaCroixDrinker can you add an integration test that verifies all 3 pieces works correctly? Thanks

10xLaCroixDrinker · 2024-04-23T00:04:26Z

@10xLaCroixDrinker can you add an integration test that verifies all 3 pieces works correctly? Thanks

Added in 18cb39d

mcollina

lgtm

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [pino](https://getpino.io) ([source](https://togithub.com/pinojs/pino)) | [`^8.20.0` -> `^8.21.0`](https://renovatebot.com/diffs/npm/pino/8.20.0/8.21.0) | [![age](https://developer.mend.io/api/mc/badges/age/npm/pino/8.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/pino/8.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/pino/8.20.0/8.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/pino/8.20.0/8.21.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>pinojs/pino (pino)</summary> ### [`v8.21.0`](https://togithub.com/pinojs/pino/releases/tag/v8.21.0) [Compare Source](https://togithub.com/pinojs/pino/compare/v8.20.0...v8.21.0) #### What's Changed - docs: add a section about pino-logfmt [#1879](https://togithub.com/pinojs/pino/issues/1879) by [@botflux](https://togithub.com/botflux) in [https://github.com/pinojs/pino/pull/1941](https://togithub.com/pinojs/pino/pull/1941) - Docs: Separate ecosystem lists by [@EstebanDalelR](https://togithub.com/EstebanDalelR) in [https://github.com/pinojs/pino/pull/1942](https://togithub.com/pinojs/pino/pull/1942) - feat: send config to transports on init by [@10xLaCroixDrinker](https://togithub.com/10xLaCroixDrinker) in [https://github.com/pinojs/pino/pull/1930](https://togithub.com/pinojs/pino/pull/1930) #### New Contributors - [@botflux](https://togithub.com/botflux) made their first contribution in [https://github.com/pinojs/pino/pull/1941](https://togithub.com/pinojs/pino/pull/1941) - [@EstebanDalelR](https://togithub.com/EstebanDalelR) made their first contribution in [https://github.com/pinojs/pino/pull/1942](https://togithub.com/pinojs/pino/pull/1942) **Full Changelog**: pinojs/pino@v8.20.0...v8.21.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/fwouts/previewjs).  Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

github-actions · 2025-04-25T00:06:41Z

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

feat: send config to transports on init

d76018c

This was referenced Mar 22, 2024

feat: post message to worker when message event is emitted pinojs/thread-stream#145

Merged

feat: load config from pino on startup pinojs/pino-abstract-transport#90

Merged

Accessing log level label from transport #1925

Closed

10xLaCroixDrinker added 2 commits April 22, 2024 13:46

Merge branch 'pinojs:master' into feature/send-config-to-transports

db7eb02

test: integration for transports using pino config

18cb39d

mcollina approved these changes Apr 24, 2024

View reviewed changes

mcollina mentioned this pull request Apr 24, 2024

Issue v9 release #1947

Merged

mcollina merged commit 5ceb596 into pinojs:master Apr 24, 2024

10xLaCroixDrinker deleted the feature/send-config-to-transports branch April 24, 2024 14:53

10xLaCroixDrinker mentioned this pull request Apr 29, 2024

feat: automatically consume pino config when using pino@>=8.21.0 pinojs/pino-pretty#508

Merged

renovate bot mentioned this pull request May 1, 2024

chore(deps): update all dependencies j:kit-282 coveo/ui-kit#3866

Merged

1 task

nioc mentioned this pull request May 2, 2024

[Config] pinoWillSendConfig leads to configuration mutation #1963

Closed

snyk-io bot mentioned this pull request May 22, 2024

[Snyk] Upgrade pino from 8.20.0 to 8.21.0 Hawthorne001/addons-linter#2

Open

This was referenced May 23, 2024

[Snyk] Upgrade pino from 8.9.0 to 8.21.0 gptkrsh/LinkFree#511

Open

[Snyk] Upgrade pino from 8.9.0 to 8.21.0 gptkrsh/LinkFree#522

Open

s-laugh mentioned this pull request May 25, 2024

[Snyk] Upgrade pino from 8.20.0 to 8.21.0 DTS-STN/passport-status#668

Closed

leika mentioned this pull request May 31, 2024

[Snyk] Upgrade pino from 8.17.2 to 8.21.0 codingfragments/tvh-guide#136

Open

davehorton mentioned this pull request Jun 6, 2024

[Snyk] Upgrade pino from 8.20.0 to 8.21.0 jambonz/sbc-sip-sidecar#64

Open

shewood mentioned this pull request Jun 7, 2024

[Snyk] Upgrade pino from 8.20.0 to 8.21.0 DTS-STN/secure-client-hub#681

Merged

thiruaws2lab mentioned this pull request Jun 12, 2024

[Snyk] Upgrade pino from 8.16.0 to 9.1.0 thiruaws2lab/10-MicroService-Appliction#2

Open

nnsutebu mentioned this pull request Jun 21, 2024

[Snyk] Upgrade pino from 8.14.1 to 8.21.0 nnsutebu/ecommerce-app-deploy-eks#4

Open

AlexOnyi mentioned this pull request Jun 30, 2024

[Snyk] Upgrade pino from 8.14.1 to 8.21.0 AlexOnyi/realworld-microservice-project#4

Open

moishimwe mentioned this pull request Jun 30, 2024

[Snyk] Upgrade pino from 8.16.0 to 8.21.0 moishimwe/realworld-microservice-project#5

Open

marcelineepi mentioned this pull request Jun 30, 2024

[Snyk] Upgrade pino from 8.16.0 to 8.21.0 marcelineepi/realworld-microservice-project#5

Open

keemraheem mentioned this pull request Jun 30, 2024

[Snyk] Upgrade pino from 8.14.1 to 8.21.0 keemraheem/realworld-microservice-project#4

Open

johanbook mentioned this pull request Jul 1, 2024

[Snyk] Upgrade pino from 8.16.1 to 8.21.0 johanbook/meet#778

Merged

fakure2020 mentioned this pull request Jul 3, 2024

[Snyk] Upgrade pino from 8.16.0 to 8.21.0 fakure2020/realworld-microservice-project01#5

Open

sawood14012 mentioned this pull request Jul 17, 2024

[Snyk] Upgrade pino from 8.5.0 to 8.21.0 redhat-appstudio/tssc-sample-templates#59

Closed

This was referenced Jul 17, 2024

[Snyk] Upgrade pino from 8.9.0 to 8.21.0 gptkrsh/LinkFree#542

Open

[Snyk] Upgrade pino from 8.9.0 to 8.21.0 gptkrsh/LinkFree#552

Open

harshakrithvikk mentioned this pull request Jul 24, 2024

[Snyk] Upgrade pino from 8.16.0 to 8.21.0 harykryshnan-Master/10-MicroService-Appliction#5

Open

gptkrsh mentioned this pull request Aug 6, 2024

[Snyk] Upgrade pino from 8.9.0 to 8.21.0 gptkrsh/LinkFree#564

Open

gptkrsh mentioned this pull request Aug 18, 2024

[Snyk] Upgrade pino from 8.9.0 to 8.21.0 gptkrsh/LinkFree#574

Open

This was referenced Sep 6, 2024

[Snyk] Upgrade pino from 8.9.0 to 8.21.0 gptkrsh/LinkFree#585

Open

[Snyk] Upgrade pino from 8.9.0 to 8.21.0 gptkrsh/LinkFree#594

Open

IT21298912 mentioned this pull request Sep 13, 2024

[Snyk] Upgrade: connect-mongo, dotenv, express-session, jsonwebtoken, nodemailer, passport, pino, pino-pretty, uuid IT21298912/Project-SPM#4

Open

sandaru-madhubhashana mentioned this pull request Sep 16, 2024

[Snyk] Upgrade: dayjs, dotenv, express, mongodb-memory-server, mongoose, nanoid, pino, pino-pretty, zod SLIIT-Y4S2/Procurement-for-Construction-Industry-SSD#2

Open

This was referenced Sep 16, 2024

[Snyk] Upgrade: , , , , , , , pino yash509/DevSecOps-10-MicroService-Appliction-GCP#1

Open

[Snyk] Upgrade: , , , , , , , async, google-protobuf, pino yash509/DevSecOps-10-MicroService-Appliction-GCP#2

Open

it21109126 mentioned this pull request Sep 18, 2024

[Snyk] Upgrade: dayjs, dotenv, mongodb-memory-server, mongoose, nanoid, pino, pino-pretty, zod SLIIT-Y4S2/Procurement-for-Construction-Industry-SSD#5

Open

Blankll mentioned this pull request Sep 18, 2024

[Snyk] Upgrade pino from 8.17.2 to 8.21.0 geek-fun/serverlessinsight#2

Closed

MalithMadusankha mentioned this pull request Sep 19, 2024

[Snyk] Upgrade: connect-mongo, dotenv, express-session, jsonwebtoken, passport, pino, pino-pretty, uuid IT21298912/SPM-new-repo#11

Open

IT21071652 mentioned this pull request Sep 26, 2024

[Snyk] Upgrade pino from 8.14.1 to 8.21.0 IT21071652/Educode-web-Application#2

Open

gyenoch mentioned this pull request Oct 3, 2024

[Snyk] Upgrade pino from 8.16.0 to 8.21.0 gyenoch/10-MicroService-Appliction#5

Open

github-actions bot locked as resolved and limited conversation to collaborators Apr 25, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: send config to transports on init#1930

feat: send config to transports on init#1930
mcollina merged 3 commits intopinojs:masterfrom
10xLaCroixDrinker:feature/send-config-to-transports

10xLaCroixDrinker commented Mar 22, 2024 •

edited

Loading

Uh oh!

mcollina commented Apr 22, 2024

Uh oh!

10xLaCroixDrinker commented Apr 23, 2024

Uh oh!

mcollina left a comment

Uh oh!

github-actions bot commented Apr 25, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

10xLaCroixDrinker commented Mar 22, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mcollina commented Apr 22, 2024

Uh oh!

10xLaCroixDrinker commented Apr 23, 2024

Uh oh!

mcollina left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Apr 25, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

10xLaCroixDrinker commented Mar 22, 2024 •

edited

Loading