GwentAPI / gwentapi

The code for GwentAPI
https://gwentapi.com
BSD 3-Clause "New" or "Revised" License
47 stars 1 forks source link

Make the logger more production friendly #15

Open Tri125 opened 7 years ago

Tri125 commented 7 years ago

Everytime something isn't found in the dB, it crested an info level entry with the logger. This is an information that warrant the debug level. We don't care about a 404, app is fine and we certainly don't need to log all of that. I'm also in favor of only loading the request tracing middleware if the debug flag has been set. I don't remember if it was required for something else or if it was just to log every requests to the router. If it was just to log everything, this should be off for prod.

Tri125 commented 7 years ago

I've look into Zap and I really like what I see. This is worth investigating.