ZeStream / zestream-server

an open-sourced alternative to cloudinary [video streaming done ✅, image/audio under dev 🔨 ]
GNU General Public License v3.0
203 stars 41 forks source link

Add Logging via Zap Logger #49

Open uzaxirr opened 1 year ago

uzaxirr commented 1 year ago

Is your feature request related to a problem? Please describe. Currently we use the traditional golang log package for logging purpose, however its not sufficient as it lacks a lot of basic functionality like an error level log.

Describe the solution you'd like Use Zap Logger and replace all the occurrence of log package with this one. Also add new logger statements whenever necessary.

Describe alternatives you've considered logrus

chauhan17nitin commented 1 year ago

Hi, @uzaxirr let me know if i can pick this,

planning to integrate a singleton Zap Logger, with TransactionID support in context,

which will ultimately help to track all logs related to a single request using a single filter on all logs.

abskrj commented 1 year ago

sure, @chauhan17nitin assigning it to you