Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add compatibility with node 0.2 #2

Open
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants

kkaefer commented Apr 15, 2011

No description provided.

@kkaefer kkaefer commented on the diff Apr 15, 2011

lib/node-syslog.js
@@ -62,7 +69,7 @@ this.Client.prototype = new(function () {
if (severity === exports.LOG_DEBUG && !this.options.debug) { return }
this.connect(function (e) {
- var pri = '<' + ((that.options.faculty * 8) + severity) + '>'; // Message priority
+ var pri = '<' + ((that.options.facility * 8) + severity) + '>'; // Message priority
@kkaefer

kkaefer Apr 15, 2011

Is this a typo? I think so. I'm seeing NaN in my syslog output.

@whiteboardmonk

whiteboardmonk Jul 21, 2012

Yup; also reported in #3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment