Suppress output

[sebastianrath]

I came across this repo through getlantern/systray. My app requires a clean stdout but the following tracing logger info is always printed:

golog/golog.go

Lines 277 to 284 in 49749a7

	l.traceOn = shouldEnableTrace(prefix)
	if l.traceOn {
	fmt.Printf("TRACE logging is enabled for prefix [%s]\n", prefix)
	l.traceOut = l.newTraceWriter()
	} else {
	fmt.Printf("TRACE logging is NOT enabled for prefix [%s]\n", prefix)
	l.traceOut = ioutil.Discard
	}

Executed through getlantern/systray/systray.go#L16:

var (
	log = golog.LoggerFor("systray")
        [...]

Happy to offer a PR. Any preferences on how to suppress the output? E.g. env var.

Best, Sebastian

[alexg-axis]

It would be nice if it could use the logger itself instead of fmt.Printf so that it can be controlled by using golog.SetOutput or golog.SetOutputs.

[alexg-axis]

Turns out that wouldn't be a solution either as, you wrote, the log is printed immediately upon inclusion. That way, there's no chance for others to configure the logging. I suggest that it's removed altogether or rewritten in a way that would allow for configuration of the logs beforehand.

Workaround:

require (
  github.com/getlantern/golog v0.0.0
  // ...
)

replace github.com/getlantern/golog => github.com/alexg-axis/golog v0.0.0-20221212110745-eb1db7fd29e5