-
Notifications
You must be signed in to change notification settings - Fork 42
Use a logger instead of puts #6
Comments
Make sure to log raw message data from Gateway and REST to make generating error logs easier. |
I think maybe we should be a little more verbose on debug. For example, we don't log REST requests, and logging of gateway messages is very inconsistent. I would like us to be able to reconstruct exactly what's happening on gateway and REST from a debug log. |
That's true, it would be more helpful. However, that being said, this section from client.cr
When logged using the logger or the many puts statements, can be extremely long for long servers (one being Discord Bots). It's not really an issue, but it would be nice to limit the log to the best that we can. |
@colinrioux if those errors are being gnerated there's either a bug in your code or a bug in discordcr and they should be reported. That's why it doesn't matter that they're long. They should be log so you notice them and report them... like it asks you. Have you reported them? |
Closing this for now as the logger is implemented; please make further issues if there are still problems |
See #6 (comment) for why this was left open |
Yes, but that is not what this issue is about ("Use a logger instead of puts"). Further logging advances could be in their own issue |
ok |
No description provided.
The text was updated successfully, but these errors were encountered: