Syslog hook doesn't log anything

[vincentbernat]

Hi!

The syslog hook doesn't log anything unless a field "level" is set by the user.

It can be corrected by using entry.Level.String() in syslog.go. However, in this case, logging is done with ANSI codes (I get something like this: #033[34mINFO#033[0m[0000] proxy: start serving requests).

[sirupsen]

Quite true. The default formatters set this. The hook then switches on the level. Did you write your own?

[vincentbernat]

I didn't set a customer formatter. I am doing that:

log.SetOutput(os.Stderr)
log.SetLevel(log.WarnLevel)
log.SetLevel(log.DebugLevel)
log.WithField("file", "something").
                Debug("main: parsing configuration file")
syslogOutput, err := logrus_syslog.NewSyslogHook("", "",
                 syslog.LOG_INFO|syslog.LOG_DAEMON, "")
if err != nil {
                  log.Fatal("main: unable to setup syslog output")
}
log.AddHook(syslogOutput)
log.WithField("listen", "something").
                Info("proxy: start serving requests")

[sirupsen]

Did you try debugging it from the hook? That seems strange.

[vincentbernat]

❦ 23 novembre 2014 07:14 -0800, Simon Eskildsen notifications@github.com :

Why are you setting the level twice?

That's a partial code. I am checking if user asked for more debug and
then check to debug. It was just to show you that I may doing
"inappropriate" stuff like start logging stuff before setting the syslog

hook.

/* Thanks to Rob `CmdrTaco' Malda for not influencing this code in any

• way.
  */
  2.4.3 linux/net/core/netfilter.c

[vincentbernat]

Here is a self contained example:

package main

import (
        log "github.com/Sirupsen/logrus"
        "github.com/Sirupsen/logrus/hooks/syslog"
        "log/syslog"
    "os"
)

func main() {
        /* Setup initial logging */
        log.SetOutput(os.Stderr)
    log.SetLevel(log.DebugLevel)
    syslogOutput, err := logrus_syslog.NewSyslogHook("", "",
        syslog.LOG_INFO|syslog.LOG_DAEMON, "")
    if err != nil {
        log.Fatal("main: unable to setup syslog output")
    }
    log.AddHook(syslogOutput)
    log.Debug("main: this should go to syslog")
}

If I leave syslog.go unchanged, nothing gets logged to syslog. It seems that this is the same problem as with #72. I'll do a pull request for that in a few minutes.

Once fixed, I still have that in syslog:

Nov 23 17:53:23 neo ./test1[31584]: #033[34mDEBU#033[0m[0000] main: this should go to syslog

I wouldn't expect the formatter to be attached only to a given output.

[sirupsen]

Fixed by #84