hipchat-appender: use previous color defaults

briceburg · briceburg · commit 21623c7947c6 · 2016-06-10T12:11:00.000-05:00
diff --git a/examples/hipchat-appender.js b/examples/hipchat-appender.js
@@ -1,4 +1,3 @@
-
 /**
  * !!! The hipchat-appender requires `hipchat-notifier` from npm
  *   - e.g. list as a dependency in your application's package.json
@@ -12,9 +11,12 @@ log4js.configure({
       "type" : "hipchat",
       "hipchat_token": "< User token with Notification Privileges >",
       "hipchat_room": "< Room ID or Name >",
-      // optionl
+      // optional
       "hipchat_from": "[ additional from label ]",
-      "hipchat_notify": "[ notify boolean to bug people ]"
+      "hipchat_notify": "[ notify boolean to bug people ]",
+      "hipchat_response_callback": function(err, response, body){
+        console.log("overridden log4js hipchat-appender response callback");
+      }
     }
   ]
 });
diff --git a/lib/appenders/hipchat.js b/lib/appenders/hipchat.js
@@ -1,4 +1,5 @@
 "use strict";
+
 var hipchat = require('hipchat-notifier');
 var layouts = require('../layouts');
 
@@ -44,6 +45,8 @@ function hipchatAppender(config, layout) {
   // @lint W074 This function's cyclomatic complexity is too high. (10)
   return function(loggingEvent){
 
+    var notifierFn;
+
     notifier.setRoom(config.hipchat_room);
     notifier.setRoom(config.hipchat_room);
     notifier.setFrom(config.hipchat_from || '');
@@ -53,11 +56,10 @@ function hipchatAppender(config, layout) {
       notifier.setHost(config.hipchat_host);
     }
 
-    var notifierFn = "success";
     switch (loggingEvent.level.toString()) {
       case "TRACE":
       case "DEBUG":
-        notifierFn = "notice";
+        notifierFn = "info";
         break;
       case "WARN":
         notifierFn = "warning";
@@ -67,7 +69,7 @@ function hipchatAppender(config, layout) {
         notifierFn = "failure";
         break;
       default:
-        notifierFn = "info";
+        notifierFn = "success";
     }
 
     // @TODO, re-work in timezoneOffset ?
