notifme / notifme-sdk

A Node.js library to send all kinds of transactional notifications.

Home Page:https://notifme.github.io/www/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Remove Winston dependency

ansarizafar opened this issue · comments

I want to replace winston with pino logger http://getpino.io/#/, which is much faster than winston. Can I use my own pino instance with notifme?

Hi @ansarizafar
It's not possible with the options, but you can override the inner instance:

const notifmeSdk = new NotifmeSdk({})
notifmeSdk.logger.innerLogger = {
  log: (level, info, extra) => { ... }
}

(log will be called here: https://github.com/notifme/notifme-sdk/blob/master/src/util/logger.js#L27)
I haven't tested but this should work (re-open if it's not the case, we'll work it out).