From 4d95fefb605cbc2cdd880c15e985691e7aacfca5 Mon Sep 17 00:00:00 2001 From: Alex Grintsvayg <grin@lbry.com> Date: Tue, 21 Apr 2020 09:25:16 -0400 Subject: [PATCH] only init sentry once --- internal/monitor/monitor.go | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/internal/monitor/monitor.go b/internal/monitor/monitor.go index 6ddf01b1..41d1729d 100644 --- a/internal/monitor/monitor.go +++ b/internal/monitor/monitor.go @@ -22,28 +22,28 @@ var textFormatter = logrus.TextFormatter{FullTimestamp: true, TimestampFormat: " // init magic is needed so logging is set up without calling it in every package explicitly func init() { configureLogger(logrus.StandardLogger()) + configureSentry(version.GetDevVersion(), mode()) +} + +func mode() string { + if config.IsProduction() { + return "production" + } else { + return "develop" + } } // configureLogger sets a few parameters for the logging subsystem. func configureLogger(l *logrus.Logger) { - var mode string - if config.IsProduction() { - mode = "production" - l.SetLevel(logrus.InfoLevel) l.SetFormatter(&jsonFormatter) } else { - mode = "develop" - l.SetLevel(logrus.TraceLevel) l.SetFormatter(&textFormatter) } - l.Infof("%s, running in %s mode", version.GetFullBuildName(), mode) - l.Infof("logging initialized (loglevel=%s)", l.Level) - - configureSentry(version.GetDevVersion(), mode) + l.Infof("%%s mode, %s level, %s", version.GetFullBuildName(), l.Level, mode()) } // LogSuccessfulQuery takes a remote method name, execution time and params and logs it