fmeringdal / nettu-meet

Open source video conferencing system for tutors.
GNU Affero General Public License v3.0
2.28k stars 301 forks source link

Logging #2

Open fmeringdal opened 3 years ago

fmeringdal commented 3 years ago

Currently all logging is done with console.log which is not good. It should be replaced with a logging library.

sha-yon commented 3 years ago

Hi @fmeringdal, would like to take this one up. Can we discuss?

fmeringdal commented 3 years ago

Sure @sayan2306 👍

yugeeklab commented 3 years ago

I would like to address this issue using "debug"

"debug" : https://github.com/visionmedia/debug

debug is a tiny debugging tool which can use in both server and frontend side.

Can I try to solve this issue?

fmeringdal commented 3 years ago

Hi @YuTaeseong !

I dont think debug library is what is needed here. The debug library is as the name implies for debugging during development. Look at this comment from one of the debug maintainers.

What I would like to see is some logging library with support for different log levels (info, error, warn, debug etc) and structured logging (more context on each log message, like timestamp, server id etc) which can be indexed into a search engine like elasticsearch. Two libraries that might have support for this is pino and winston if you would like to have a look at them.