[core] Fix correct name value on the transport

It must be `syslog` and not `Syslog`.
winstonjs · Dec 22, 2015 · 2750b51 · 2750b51
1 parent 72ee625
commit 2750b51
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/lib/winston-syslog.js b/lib/winston-syslog.js
@@ -34,8 +34,6 @@ var levels = Object.keys({
 var Syslog = exports.Syslog = function (options) {
   winston.Transport.call(this, options);
   options = options || {};
-  // Set transport name
-  this.name = 'syslog';
   //
   // Setup connection state
   //
@@ -108,7 +106,7 @@ winston.transports.Syslog = Syslog;
 //
 // Expose the name of this Transport on the prototype
 //
-Syslog.prototype.name = 'Syslog';
+Syslog.prototype.name = 'syslog';
 //
 // ### function log (level, msg, [meta], callback)
 // #### @level {string} Target level to log to

diff --git a/test/syslog-test.js b/test/syslog-test.js
@@ -89,6 +89,14 @@ vows.describe('winston-syslog').addBatch({
         assert.equal(transport.localhost, false);
         transport.close();
       }
+    },
+    'adding / removing transport to syslog': {
+      'should just work': function () {
+        winston.add(winston.transports.Syslog);
+        winston.remove(winston.transports.Syslog);
+        winston.add(winston.transports.Syslog);
+        winston.remove(winston.transports.Syslog);
+      }
     }
   }
 }).export(module);