Skip to content

Commit

Permalink
Abstract logging for the middleware (#96)
Browse files Browse the repository at this point in the history
* Move dedupe code to dedupe.go

* Add unified logging interface.

* Adapt middleware/, user/, signals/ and server/ to new logging interfaces.

Allow user to specify logging implementation in server, if non is specified default to logrus.
As part of this we need log level as server config, and for convenience we expose the server logger.

* Update vendoring

* Expose the various level types.

Signed-off-by: Tom Wilkie <[email protected]>
  • Loading branch information
tomwilkie authored and bboreham committed Jul 19, 2018
1 parent 5d6891c commit 54f55b1
Show file tree
Hide file tree
Showing 14 changed files with 554 additions and 219 deletions.
72 changes: 50 additions & 22 deletions Gopkg.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

137 changes: 137 additions & 0 deletions logging/dedupe.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,137 @@
package logging

import (
"fmt"
"strings"
"sync"
"time"

log "github.com/sirupsen/logrus"
)

const (
defaultDedupeInterval = time.Minute
)

// SetupDeduplication should be performed after any other logging setup.
// For all logs less severe or equal to the given log level (but still higher than the logger's configured log level),
// these logs will be 'deduplicated'. What this means is that, excluding certain special fields like time, multiple
// identical log entries will be grouped up and a summary message emitted.
// For example, instead of:
// 00:00:00 INFO User 123 did xyz
// 00:00:10 INFO User 123 did xyz
// 00:00:25 INFO User 123 did xyz
// 00:00:55 INFO User 123 did xyz
// you would get:
// 00:00:00 INFO User 123 did xyz
// 00:01:00 INFO Repeated 3 times: User 123 did xyz
// The interval argument controls how long to wait for additional messages to arrive before reporting.
// Increase it to deduplicate more aggressively, decrease it to lower latency from a log occurring to it appearing.
// Set it to 0 to pick a sensible default value (recommended).
// NOTE: For simplicity and efficiency, fields are considered 'equal' if and only if their string representations (%v) are equal.
func SetupDeduplication(logLevel string, interval time.Duration) error {
dedupeLevel, err := log.ParseLevel(logLevel)
if err != nil {
return fmt.Errorf("Error parsing log level: %v", err)
}
if interval <= 0 {
interval = defaultDedupeInterval
}

// We use a special Formatter to either format the log using the original formatter, or to return ""
// so nothing will be written for that event. The repeated entries are later logged along with a field flag
// that tells the formatter to ignore the message.
stdLogger := log.StandardLogger()
stdLogger.Formatter = newDedupeFormatter(stdLogger.Formatter, dedupeLevel, interval)
return nil
}

type entryCount struct {
entry log.Entry
count int
}

type dedupeFormatter struct {
innerFormatter log.Formatter
level log.Level
interval time.Duration
seen map[string]entryCount
lock sync.Mutex
}

func newDedupeFormatter(innerFormatter log.Formatter, level log.Level, interval time.Duration) *dedupeFormatter {
return &dedupeFormatter{
innerFormatter: innerFormatter,
level: level,
interval: interval,
seen: map[string]entryCount{},
}
}

func (f *dedupeFormatter) Format(entry *log.Entry) ([]byte, error) {
if f.shouldLog(entry) {
b, err := f.innerFormatter.Format(entry)
return b, err
}
return []byte{}, nil
}

func (f *dedupeFormatter) shouldLog(entry *log.Entry) bool {
if _, ok := entry.Data["deduplicated"]; ok {
// ignore our own logs about deduped messages
return true
}
if entry.Level < f.level {
// ignore logs more severe than our level
return true
}
key := fmt.Sprintf("%s %s", entry.Message, fieldsToString(entry.Data))
f.lock.Lock()
defer f.lock.Unlock()
if ec, ok := f.seen[key]; ok {
// already seen, increment count and do not log
ec.count++
f.seen[key] = ec
return false
}
// New message, log it but add it to seen.
// We need to copy because the pointer ceases to be valid after we return from Format
f.seen[key] = entryCount{entry: *entry}
go f.evictEntry(key) // queue to evict later
return true
}

// Wait for interval seconds then evict the entry and send the log
func (f *dedupeFormatter) evictEntry(key string) {
time.Sleep(f.interval)
var ec entryCount
func() {
f.lock.Lock()
defer f.lock.Unlock()
ec = f.seen[key]
delete(f.seen, key)
}()
if ec.count == 0 {
return
}
entry := log.WithFields(ec.entry.Data).WithField("deduplicated", ec.count)
message := fmt.Sprintf("Repeated %d times: %s", ec.count, ec.entry.Message)
// There's no way to choose the log level dynamically, so we have to do this hack
map[log.Level]func(args ...interface{}){
log.PanicLevel: entry.Panic,
log.FatalLevel: entry.Fatal,
log.ErrorLevel: entry.Error,
log.WarnLevel: entry.Warn,
log.InfoLevel: entry.Info,
log.DebugLevel: entry.Debug,
}[ec.entry.Level](message)
}

func fieldsToString(data log.Fields) string {
parts := make([]string, 0, len(data))
// traversal order here is arbitrary but stable, which is fine for our purposes
for k, v := range data {
parts = append(parts, fmt.Sprintf("%s=%v", k, v))
}
return strings.Join(parts, " ")
}
58 changes: 58 additions & 0 deletions logging/global.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,58 @@
package logging

var global Interface = Noop()

// Global returns the global logger.
func Global() Interface {
return global
}

// SetGlobal sets the global logger.
func SetGlobal(i Interface) {
global = i
}

// Debugf convenience function calls the global loggerr.
func Debugf(format string, args ...interface{}) {
global.Debugf(format, args...)
}

// Debugln convenience function calls the global logger.
func Debugln(args ...interface{}) {
global.Debugln(args...)
}

// Infof convenience function calls the global logger.
func Infof(format string, args ...interface{}) {
global.Infof(format, args...)
}

// Infoln convenience function calls the global logger.
func Infoln(args ...interface{}) {
global.Infoln(args...)
}

// Warnf convenience function calls the global logger.
func Warnf(format string, args ...interface{}) {
global.Warnf(format, args...)
}

// Warnln convenience function calls the global logger.
func Warnln(args ...interface{}) {
global.Warnln(args...)
}

// Errorf convenience function calls the global logger.
func Errorf(format string, args ...interface{}) {
global.Errorf(format, args...)
}

// Errorln convenience function calls the global logger.
func Errorln(args ...interface{}) {
global.Errorln(args...)
}

// WithField convenience function calls the global logger.
func WithField(key string, value interface{}) Interface {
return global.WithField(key, value)
}
Loading

0 comments on commit 54f55b1

Please sign in to comment.