search

log4mongo / log4mongo-java

log4j appender to MongoDB database
http://log4mongo.org
Apache License 2.0
80 stars 29 forks source link

Implemented compatibility with log4j 1.2.14 #6

Closed padcom closed 12 years ago

padcom commented 12 years ago

The current implementation uses getTimeStamp() method not available in older version of log4j. To use it with 1.2.14 for example one needs to switch to field access instead of method access to get the timestamp information

padcom commented 12 years ago

We're trying to implement the log4mongo appender within our jboss-4.2.3 application. That prehistoric version of JBoss uses log4j 1.2.14 which lacks the getTimeStamp method but instead provides a timeStamp field present in both 1.2.14 as well as in 1.2.16. It'd be great if we could make it backwards-compatible.

RobertStewart commented 12 years ago

I will take a look at your pull request soon. I just pushed another change that I think complicates things for you. The MDC enhancement takes advantage of a feature that I believe was added in log4j 1.2.15. I believe I can workaround that, though. I seem to remember some other reason for having gone with 1.2.16. Will have to dig through old commit comments to see if I can refresh my memory.

padcom commented 12 years ago

For what it's worth for now (before the upgrade to 7.1) we've created a separate instance of Log4J for the application itself and this fix is not needed anymore. So feel free to discard this pull request.

castlight commented 12 years ago

Thanks for the update, Matthias!