sseserver icon indicating copy to clipboard operation
sseserver copied to clipboard

move off deprecated azer/Debug dependency for debugging solution

Open mroth opened this issue 6 years ago • 2 comments

Some options:

  • https://github.com/azer/logger
  • https://github.com/rs/zerolog
  • https://github.com/sirupsen/logrus (thx @nerney)
  • https://github.com/uber-go/zap

Logging will need to be documented in the README as well, since it's essential for someone who would be using this package in production.

mroth avatar Jul 13 '18 15:07 mroth

another good option: https://github.com/sirupsen/logrus (it's what docker uses)

nerney avatar Jul 13 '18 17:07 nerney

With the changes in 5338305, a logging solution is no longer required, as it can be left up to the end implementation, as is probably more appropriate. Debug is now only used for actual DEBUGGING in development.

That said, it may be worth pulling Debug out of the codebase since it's deprecated. Changing the name of this Issue according.

mroth avatar Jul 21 '18 00:07 mroth