t0xa / gelfj

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

Log4j backwards compatibility #1

Closed nstielau closed 13 years ago

nstielau commented 13 years ago

Hey,

Getting ready to try this out. Talking with joemiller on #graylog2, gelfj requires Log4j 1.2.16. Here's his quick fix for backwards compatibility, which could possibly be determined programmatically and incorporated into gelfj:

+++ b/src/main/java/org/graylog2/log/GelfAppender.java @@ -64,7 +64,9 @@ public class GelfAppender extends AppenderSkeleton { @Override protected void append(LoggingEvent event) { Level level = event.getLevel();

t0xa commented 13 years ago

Hi,

Thanks for suggestion. I've committed fix with detection of a Log4j version and uploaded v0.3 of a jar. Could you please check from you side?

Anton