seneca icon indicating copy to clipboard operation
seneca copied to clipboard

Can't use Pino logger with seneca-pino-logger adapter after [email protected]

Open daniegarcia254 opened this issue 6 years ago • 1 comments

We can't start our app because Pino logger with senca-pino-logger adapter can't be use any longer after changes made in [email protected].

I'v opened an issue in https://github.com/blueshirts/seneca-pino-adapter/issues/2 where you can see the error we're receiving, but still, you could maybe point the direction of the issue and propose a solution.

const Pino = require('pino');
const Noir = require('pino-noir');
const SenecaPinoLogger = require('seneca-pino-adapter');

const logger = new SenecaPinoLogger({
        logger: Pino({
            config: {
              serializers: Noir(require('./redactions.js'), undefined)
              , name: 'SenecaLogger'
              , level: 'info'
              , prettyPrint: { colorize: true }
            }
        })
    });

const seneca = require ('seneca') ({
    ....
    , internal: { logger: logger }
    , legacy: {
        logging: false
    }
})

We are using:

daniegarcia254 avatar Sep 12 '19 06:09 daniegarcia254

I've got a PR that fixes this: https://github.com/blueshirts/seneca-pino-adapter/pull/3

Hopefully @blueshirts can merge that & publish a fixed version.

tswaters avatar Oct 07 '19 16:10 tswaters