Add logging options to config

This commit is contained in:
Geert-Johan Riemer 2014-01-30 13:47:28 +01:00
parent be2914fd12
commit 0c51ff11d1
1 changed files with 38 additions and 12 deletions

44
tail.go
View File

@ -47,6 +47,10 @@ type Config struct {
// Generic IO // Generic IO
Follow bool // Continue looking for new lines (tail -f) Follow bool // Continue looking for new lines (tail -f)
MaxLineSize int // If non-zero, split longer lines into multiple lines MaxLineSize int // If non-zero, split longer lines into multiple lines
// Logging
DisableLogging bool // when true, disables all logging (i.e. Config.Logger isn't used)
Logger *log.Logger // when not specified (nil), this will be set to `log.New(os.Stderr, "", log.LstdFlags)`
} }
type Tail struct { type Tail struct {
@ -75,7 +79,13 @@ func TailFile(filename string, config Config) (*Tail, error) {
t := &Tail{ t := &Tail{
Filename: filename, Filename: filename,
Lines: make(chan *Line), Lines: make(chan *Line),
Config: config} Config: config,
}
// when Logger was not specified in config, use default logger
if t.Logger == nil {
t.Logger = log.New(os.Stderr, "", log.LstdFlags)
}
t.rateMon = new(RateMonitor) t.rateMon = new(RateMonitor)
@ -135,7 +145,9 @@ func (tail *Tail) reopen() error {
tail.file, err = os.Open(tail.Filename) tail.file, err = os.Open(tail.Filename)
if err != nil { if err != nil {
if os.IsNotExist(err) { if os.IsNotExist(err) {
log.Printf("Waiting for %s to appear...", tail.Filename) if !tail.DisableLogging {
tail.Logger.Printf("Waiting for %s to appear...", tail.Filename)
}
if err := tail.watcher.BlockUntilExists(&tail.Tomb); err != nil { if err := tail.watcher.BlockUntilExists(&tail.Tomb); err != nil {
if err == tomb.ErrDying { if err == tomb.ErrDying {
return err return err
@ -174,7 +186,9 @@ func (tail *Tail) tailFileSync() {
// Seek to requested location on first open of the file. // Seek to requested location on first open of the file.
if tail.Location != nil { if tail.Location != nil {
_, err := tail.file.Seek(tail.Location.Offset, tail.Location.Whence) _, err := tail.file.Seek(tail.Location.Offset, tail.Location.Whence)
// log.Printf("Seeked %s - %+v\n", tail.Filename, tail.Location) if !tail.DisableLogging { //
tail.Logger.Printf("Seeked %s - %+v\n", tail.Filename, tail.Location)
}
if err != nil { if err != nil {
tail.Killf("Seek error on %s: %s", tail.Filename, err) tail.Killf("Seek error on %s: %s", tail.Filename, err)
return return
@ -257,24 +271,34 @@ func (tail *Tail) waitForChanges() error {
tail.changes = nil tail.changes = nil
if tail.ReOpen { if tail.ReOpen {
// XXX: we must not log from a library. // XXX: we must not log from a library.
log.Printf("Re-opening moved/deleted file %s ...", tail.Filename) if !tail.DisableLogging {
tail.Logger.Printf("Re-opening moved/deleted file %s ...", tail.Filename)
}
if err := tail.reopen(); err != nil { if err := tail.reopen(); err != nil {
return err return err
} }
log.Printf("Successfully reopened %s", tail.Filename) if !tail.DisableLogging {
tail.Logger.Printf("Successfully reopened %s", tail.Filename)
}
tail.reader = bufio.NewReader(tail.file) tail.reader = bufio.NewReader(tail.file)
return nil return nil
} else { } else {
log.Printf("Stopping tail as file no longer exists: %s", tail.Filename) if !tail.DisableLogging {
tail.Logger.Printf("Stopping tail as file no longer exists: %s", tail.Filename)
}
return ErrStop return ErrStop
} }
case <-tail.changes.Truncated: case <-tail.changes.Truncated:
// Always reopen truncated files (Follow is true) // Always reopen truncated files (Follow is true)
log.Printf("Re-opening truncated file %s ...", tail.Filename) if !tail.DisableLogging {
tail.Logger.Printf("Re-opening truncated file %s ...", tail.Filename)
}
if err := tail.reopen(); err != nil { if err := tail.reopen(); err != nil {
return err return err
} }
log.Printf("Successfully reopened truncated %s", tail.Filename) if !tail.DisableLogging {
tail.Logger.Printf("Successfully reopened truncated %s", tail.Filename)
}
tail.reader = bufio.NewReader(tail.file) tail.reader = bufio.NewReader(tail.file)
return nil return nil
case <-tail.Dying(): case <-tail.Dying():
@ -300,10 +324,12 @@ func (tail *Tail) sendLine(line []byte) bool {
tail.Lines <- &Line{line, now, nil} tail.Lines <- &Line{line, now, nil}
rate := tail.rateMon.Tick(nowUnix) rate := tail.rateMon.Tick(nowUnix)
if tail.LimitRate > 0 && rate > tail.LimitRate { if tail.LimitRate > 0 && rate > tail.LimitRate {
log.Printf("Rate limit (%v < %v) reached on file (%v); entering 1s cooloff period.\n", if !tail.DisableLogging {
tail.Logger.Printf("Rate limit (%v < %v) reached on file (%v); entering 1s cooloff period.\n",
tail.LimitRate, tail.LimitRate,
rate, rate,
tail.Filename) tail.Filename)
}
return false return false
} }
} }