Chaosthebot / Chaos

A social coding experiment that updates its own code democratically.
http://chaosthebot.com
MIT License
2.44k stars 210 forks source link

Too much logging #440

Closed mark-i-m closed 7 years ago

mark-i-m commented 7 years ago

We need to cut down on logging HTTP requests. They take up too much space in the log and aren't that useful...

Swizz commented 7 years ago

Log result of only 1 request

06-01 07:38 github_api   INFO     requesting GET to '/repos/chaosbot/Chaos/issues/450/comments'
06-01 07:38 urllib3.connectionpool DEBUG    Starting new HTTPS connection (1): api.github.com
06-01 07:38 urllib3.connectionpool DEBUG    https://api.github.com:443 "GET /repos/chaosbot/Chaos/issues/450/comments?per_page=100 HTTP/1.1" 200 None
06-01 07:38 github_api   DEBUG    requests remaining: 2958, reset in: 2308s, cooldown sleep: 0.05s

Could be refactored to :

06-01 07:38 github_api   INFO     requesting GET to '/repos/chaosbot/Chaos/issues/450/comments'
06-01 07:38 github_api   DEBUG    requests remaining: 2958, reset in: 2308s, cooldown sleep: 0.05s
mark-i-m commented 7 years ago

Should be improved by #477

mark-i-m commented 7 years ago

/vote close

chaosbot commented 7 years ago

/vote close

Command Ran

chaosbot commented 7 years ago

:no_entry: The issue has been closed after a vote.

chaosbot commented 7 years ago

/vote close

Command Ran

chaosbot commented 7 years ago

:no_entry: The issue has been closed after a vote.