[Logger] What do you think about changing "winston" to "pino"

TheSoftwareHouse / node-common

Common libraries used by TSH

MIT License

13 stars 4 forks source link

[Logger] What do you think about changing "winston" to "pino" #35

Closed RunAge closed 2 years ago

RunAge commented 2 years ago

Pros:

Build in Redaction / Obfuscation
Log processing work on separate thread
JSON logs with separate package to get nice readable ones pino-pretty
Faster

Cons:

Some work needed to create logs with similar style

sethii commented 2 years ago

So far we don't see logger performance to be an issue and so we don't really see the benefits. Just recently we've being doing some internal research about different loggers and at the end we decided to stay with winston, because there was no noticeable difference in performance and response time.