Merge pull request #21 from kkamkou/issue-20

Do not send an empty message, closes #20

.gitignore

@@ -1,3 +1,4 @@
 /.idea/
 /node_modules/
 /coverage/
+*.log

lib/gelf-pro.js

@@ -130,6 +130,8 @@ gelf.send = function (message, cb) {
  * @param {Function} cb
  */
 gelf.message = function (message, lvl, extra, cb) {
+  if (_.isNil(message)) { return; }
+
   // it is possible to skip the extra variable
   if (_.isFunction(extra) && !cb) {
     cb = extra;

package.json

@@ -1,6 +1,6 @@
 {
   "name": "gelf-pro",
-  "version": "0.8.1",
+  "version": "0.8.2",
   "main": "./lib/gelf-pro.js",
   "author": "Kanstantsin Kamkou <[email protected]>",
   "description": "The Graylog Extended Log Format for the Node.js",
@@ -17,7 +17,7 @@
     "test": "./node_modules/.bin/istanbul cover ./node_modules/mocha/bin/_mocha --report lcovonly -- -R spec && cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js && rm -rf ./coverage"
   },
   "dependencies": {
-    "lodash": "~4.14",
+    "lodash": "~4.15",
     "async": "~2.0"
   },
   "devDependencies": {

test/tests.js

@@ -124,6 +124,26 @@ module.exports = {
 
     },
 
+    'Avoid an empty message': function (done) {
+      var gelf = _.cloneDeep(gelfOriginal);
+      sinon.spy(gelf, 'send');
+      gelf.message();
+      process.nextTick(function () {
+        gelf.send.calledOnce.should.be.false();
+        done();
+      });
+    },
+
+    'Avoid a message with null': function (done) {
+      var gelf = _.cloneDeep(gelfOriginal);
+      sinon.spy(gelf, 'send');
+      gelf.message();
+      process.nextTick(function () {
+        gelf.send.calledOnce.should.be.false();
+        done();
+      });
+    },
+
     'Work with dates': function () {
       var gelf = _.cloneDeep(gelfOriginal),
         date = new Date();