Changeset 747 in code for trunk/server.go

Timestamp:

Dec 6, 2021, 5:35:49 PM (4 years ago)

Author:

contact

Message:

Add Logger.Debugf

Makes it easy to print debugging messages which aren't targeted at
the user. See [1] for motivation.

[1]: ​https://dave.cheney.net/2015/11/05/lets-talk-about-logging

File:

• trunk/server.go (modified) (4 diffs)

trunk/server.go

@@ -39 +39 @@
 type Logger interface {
         Printf(format string, v ...interface{})
+        Debugf(format string, v ...interface{})
+}
+
+type logger struct {
+        *log.Logger
+        debug bool
+}
+
+func (l logger) Debugf(format string, v ...interface{}) {
+        if !l.debug {
+                return
+        }
+        l.Logger.Printf(format, v...)
+}
+
+func NewLogger(out io.Writer, debug bool) Logger {
+        return logger{
+                Logger: log.New(log.Writer(), "", log.LstdFlags),
+                debug:  debug,
+        }
 }
 
@@ -51 +71 @@
         v = append([]interface{}{l.prefix}, v...)
         l.logger.Printf("%v"+format, v...)
+}
+
+func (l *prefixLogger) Debugf(format string, v ...interface{}) {
+        v = append([]interface{}{l.prefix}, v...)
+        l.logger.Debugf("%v"+format, v...)
 }
 
@@ -73 +98 @@
         Title           string
         LogPath         string
-        Debug           bool
         HTTPOrigins     []string
         AcceptProxyIPs  config.IPSet
@@ -110 +134 @@
 func NewServer(db Database) *Server {
         srv := &Server{
-                Logger:    log.New(log.Writer(), "", log.LstdFlags),
+                Logger:    NewLogger(log.Writer(), true),
                 db:        db,
                 listeners: make(map[net.Listener]struct{}),