Fixes #24. No longer throw exception when object is logged. Added uni…

…t test.
log4mongo · Jan 9, 2015 · 7a70433 · 7a70433
1 parent aecbfe3
commit 7a70433
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/src/main/java/org/log4mongo/LoggingEventBsonifierImpl.java b/src/main/java/org/log4mongo/LoggingEventBsonifierImpl.java
@@ -98,7 +98,7 @@ public DBObject bsonify(final LoggingEvent loggingEvent) {
             result.put(KEY_TIMESTAMP, new Date(loggingEvent.getTimeStamp()));
             nullSafePut(result, KEY_LEVEL, loggingEvent.getLevel().toString());
             nullSafePut(result, KEY_THREAD, loggingEvent.getThreadName());
-            nullSafePut(result, KEY_MESSAGE, loggingEvent.getMessage());
+            nullSafePut(result, KEY_MESSAGE, loggingEvent.getRenderedMessage());
             nullSafePut(result, KEY_LOGGER_NAME, bsonifyClassName(loggingEvent.getLoggerName()));
 
             addMDCInformation(result, loggingEvent.getProperties());

diff --git a/src/test/java/org/log4mongo/TestMongoDbAppender.java b/src/test/java/org/log4mongo/TestMongoDbAppender.java
@@ -290,6 +290,18 @@ public void testHostInfoRecords() throws Exception {
         assertEquals(ManagementFactory.getRuntimeMXBean().getName(), hostinfo.get("process"));
     }
 
+    @Test
+    /**
+     * Added to verify fix to GitHub issue #24.
+     */
+    public void testLogObject() throws Exception {
+        Object object = new Object();
+
+        assertEquals(0L, countLogEntries());
+        log.error(object);
+        assertEquals(1L, countLogEntries());
+    }
+
     private long countLogEntries() {
         return (collection.getCount());
     }