t0xa / gelfj

Graylog Extended Log Format (GELF) implementation in Java and log4j appender without any dependencies.
https://github.com/t0xa/gelfj/wiki
Other
186 stars 116 forks source link

GelfHandler does not map log4j log levels correctly #60

Open zpenzeli opened 11 years ago

zpenzeli commented 11 years ago

If I'm using the GelfHandler, all logs are reported as "INFO", because they don't have the proper mapping to Syslog levels.

I made a "quick fix" in 1741c9b19b1db9be495803be3c41dea73d43d661 (in my forked repo), but my problems with that:

  1. It's only a fix for log4j, I'm not familiar with other logging libraries, like slf4j, etc.
  2. Unit test not yet provided, I have to figure out how to do it properly.

Please comment if you have some time...

Regards,

Zoltan

t0xa commented 11 years ago

Hello Zoltan,

Many thanks for sharing the fix. I'm trying now to deal with the backlog of issues / pull rqsts and will get back asa after that.

Have a great day, Anton

rthaenert commented 8 years ago

@zpenzeli , @t0xa Wasn't this already fixed? We had similar problems (everything was reported the default level 7) with version 1.0.2 but the mapping works correctly with version 1.1.15