GoodDollar/GoodServer

switch logger from pino

Closed this issue · 2 comments

sirpy commented

pino doesnt allow to override the .error method
so we cant send errors also to 3rd apis

find a different logger, maybe winston?

@sirpy I found the way to extend the error method of pino logger. Working with child loggers too.
You can find the implementation in this PR GoodDollar/GoodDAPP#1055 (commit - GoodDollar/GoodDAPP@86e365e).

sirpy commented

@AlexeyKosinski @yaroslav-fedyshyn-nordwhale this is a GoodServer issue.
on client it we already overwrite the error logging in analytics.js, update your GoodDAPP PR.
but regardless the issue is on server.