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?
yaroslav-fedyshyn-nordwhale commented
@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.