gateio-crypto-trading-bot-binance-announcements-new-coins icon indicating copy to clipboard operation
gateio-crypto-trading-bot-binance-announcements-new-coins copied to clipboard

move regular logging to debug

Open smuu opened this issue 3 years ago • 3 comments

As the log is 'spammed' by two regular tasks, the log gets big pretty fast. Also, as an end user, I expect that the basic future works and not get logged in info but in debug. This is a proposal as an alternative to the approach in #41.

Should we consider alternate the logging behavior on any other parts of the application?

smuu avatar Oct 25 '21 11:10 smuu

Another good idea is https://github.com/CyberPunkMetalHead/gateio-crypto-trading-bot-binance-announcements-new-coins/pull/74. I'd suggest leaving it on info since that's the best fit. It's not really a debug statement.

DominicFrei avatar Nov 15 '21 01:11 DominicFrei

Another good idea is #74. I'd suggest leaving it on info since that's the best fit. It's not really a debug statement.

After thinking about that again, I stand by the idea of it not being debug but I guess it's also not info. It's just a heartbeat that clutters the log. Maybe remove it altogether and only log problems and actual announcements and deal with the heartbeat differently? We had a discussion about that in Discord that I thought was really productive.

DominicFrei avatar Nov 16 '21 23:11 DominicFrei

I agree, I would remove it altogether and prioritize error logging.

Linus045 avatar Dec 28 '21 18:12 Linus045