nestjs-pino icon indicating copy to clipboard operation
nestjs-pino copied to clipboard

Out of date example on README for `pinoHttp` option

Open micalevisk opened this issue 2 years ago • 3 comments

https://github.com/iamolegga/nestjs-pino/blob/18c701066d8ff38af5511f3665b9b3de511891f9/README.md#L229-L241

image

What does works:

    LoggerModule.forRoot({
      pinoHttp:
        {
          logger: pino({
            useLevelLabels: true,
            level: process.env.NODE_ENV !== 'production' ? 'debug' : 'info',
            transport: process.env.NODE_ENV !== 'production'
              ? { target: 'pino-pretty' }
              : undefined,
          })
        },
    })

but I believe the semantics here isn't the same

micalevisk avatar Mar 18 '22 23:03 micalevisk

useLevelLabels does not work for me , use code @micalevisk provided will initialize another logger instead of default logger, in my case I only specified a file stream it results to stdout and file.

bung87 avatar Mar 23 '22 14:03 bung87

Looks like new syntax is:

    LoggerModule.forRoot({
      pinoHttp: {
        level: process.env.NODE_ENV !== 'production' ? 'debug' : 'info',
        transport:
          process.env.NODE_ENV !== 'production'
            ? { target: 'pino-pretty' }
            : undefined,
      },
    }),

LeoMartinDev avatar Apr 13 '22 09:04 LeoMartinDev

New update for replacing log level value to level label. Example:

{"level": 20} => {"level":"debug"}

LoggerModule.forRoot({
  pinoHttp: {
    logger: pino({
      formatters: {
        level: (label) => {
          return { level: label }
        },
      },
      level: process.env.NODE_ENV !== 'production' ? 'debug' : 'info',
    }),
    transport:
      process.env.NODE_ENV !== 'production'
        ? { target: 'pino-pretty' }
        : undefined,
  },
})

lekhanhtoan37 avatar Jun 23 '22 19:06 lekhanhtoan37