chris-rock / bunyan-logstash-tcp

logstash tcp module for bunyan
21 stars 21 forks source link

Update logstash.js #19

Closed jfstgermain closed 7 years ago

jfstgermain commented 9 years ago

Checking if socket.unref exists. In node 0.10 the CleartextStream object returned by tls.connect doesn't have an unref() function

See https://github.com/chris-rock/bunyan-logstash-tcp/issues/18

rossille commented 7 years ago

hi @jfstgermain First, thank you for this pull request and sorry for the late answer. I'm taking over management of this lib, since the author doesn't have the time to do it anymore. I had to move the projet to https://github.com/transcovo/bunyan-logstash-tcp and add tests before being able to accept pull requests. I think that this pull request is not needed anymore since I dropped the support of node 0.x in the 1.0.0 version, but please know that: