James Sumners

I'd be okay with documentation that describes scenarios on how to use it and log it. But I'm not keen on forcing this with fatal logs. While the _recommendation_ is...

Sure.

Indeed. I’m suggesting keeping what we currently have and extending it with a version of what is linked.

Yeah. This issue is a TODO for me in the future or easy fodder for a new contributor.

> It seems to only fail when run using jest & supertest though. I know nothing of these tools, but I'm willing to bet several dollars that they modify `stdout`.

PRs are welcome. But I think this is already covered by the docs that say the final logging requires streams with file descriptors. I think a more appropriate doc addition...

I would be fine with it.

Simply going on the information linked, I really don't see the issue. It was never a nation. But I don't really care about the TLD. I only suggest that it...

The default request serializer is https://github.com/pinojs/pino-std-serializers . This serializer is extensible and thus exposes information necessary for downstream tools to do their work. Any implementation that doesn't extend that serializer...

Pino does not log the line at which the log method was invoked. https://github.com/pinojs/pino-caller adds such functionality in Node, but I don't know about the browser.