indexzero/winston-syslog — pull request #3 by thurmda (2012-02-13 21:00:27+00:00)
Title: Could you please review changes to winston-syslog.

I made some changes that provide my use case the expected functionality, however I don't understand the majority of the connect method. I provided inline comments accompanying the commit in github. I also added nodeunit test simply because i'm unsure how to do the same with vows.

Here are a few items that I just don't get about the connect method:
1) If syslogd is listening on localhost:514 how would this socket ever bind?
2) If you want to send messages why would you bind and listen on a port?
3) What's the point of binding and not handling 'message' events?

PS If I didn't follow coding conventions It's simply an oversight, just point me in the right direction and I'll update.