flush & flushSync stdout on Node core streams

[ronag]

Even in non extreme mode logs are not properly flushed during process.exit. See, nodejs/node#6456

No sure how to resolve this? Should we send a cb to every write call to keep a isFlushed flag up to date?

[davidmarkclements]

is this relevant to the v5 release and the sonic-boom approach (cc @mcollina)

[ronag]

The sonic-boom approach does it correctly. This is more of a problem with the non extreme/sonic-boom approach. We don't have a way to flush stdout.

[davidmarkclements]

yeah that's a node core issue - so it's a wont-fix for us - we recommend using pino.destination in the v5 docs and v5 use pino.destination(1) by default to supply stdout flushing

[ronag]

@davidmarkclements Fair enough. Though we might want to emphasize this in the docs?

[davidmarkclements]

absolutely - feel welcome to PR against next-major docs

[github-actions]

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