[QUESTION] Why `PinoLogger` instance don't show the error object about nestjs-pino HOT 6 CLOSED

Good catch, thanks. I'll try to see what can be done here

from nestjs-pino.

Pino expects Error object in err field and can correctly serialize it.

Example:

this.logger.error({ err: new Error('message') })

from nestjs-pino.

Hi, it's interesting, could you please provide minimal example repo on this?

from nestjs-pino.

While testing the minimal example I figured out that this issue appear when I define the context to the PinoLogger instance. Without calling logger.setContext or using the @InjectPinoLogger decorator, this.logger.error(Error('foo')) show the error object.

You can download the example here: https://codesandbox.io/s/nestjs-pino-issue-304-lgxwv

from nestjs-pino.

Added extra check if passing instance of Error, and if so transfering it to err field. Anyway that's more related to pino than to this lib. if you passing some custom error please check this out: https://github.com/Microsoft/TypeScript-wiki/blob/master/Breaking-Changes.md#extending-built-ins-like-error-array-and-map-may-no-longer-work. Also this issue might be useful too: pinojs/pino#579

shipped in v1.3.0

from nestjs-pino.

I just realize this is a bug :/

This happen due to the following code

from nestjs-pino.