From ca635fc4b99af217e24d6b865499e8f298dd0d82 Mon Sep 17 00:00:00 2001 From: Jens John Date: Thu, 10 Jul 2014 12:17:35 +0900 Subject: [PATCH] GELF appender: don't send full_message field According to the GELF spec v1.1 [1], the full_message field in GELF is optional. The log4js implemention until now has sent identitical short_message and full_message fields. Since this does not add any new information to the log message, I suggest that full_message be dropped from GELF. -- [1] http://graylog2.org/gelf#specs --- lib/appenders/gelf.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lib/appenders/gelf.js b/lib/appenders/gelf.js index 8e075b5..e3df4f1 100644 --- a/lib/appenders/gelf.js +++ b/lib/appenders/gelf.js @@ -96,8 +96,7 @@ function gelfAppender (layout, host, port, hostname, facility) { function preparePacket(loggingEvent) { var msg = {}; addCustomFields(loggingEvent, msg); - msg.full_message = layout(loggingEvent); - msg.short_message = msg.full_message; + msg.short_message = layout(loggingEvent); msg.version="1.0"; msg.timestamp = msg.timestamp || new Date().getTime() / 1000; // log should use millisecond