search

loggly / node-loggly-bulk

A client implementation for Loggly in node.js
https://www.loggly.com/docs/node-js-logs/
Other
20 stars 19 forks source link

Fix for Issue #31 #33

Closed blyork closed 6 years ago

blyork commented 6 years ago

Fix for Issue #31.

Shwetajain148 commented 6 years ago

Hi @blyork, First of all, thank you for looking into this issue and giving your time on that to debug the library. I tested your code at my end in a continue event logging environment and after successful logging of near about 2 hours I could see the same callback error in the library.

cc: @mostlyjason

Shwetajain148 commented 6 years ago

Hi @blyork, The same fix was made in the library and the callback issue was also resolved. Can you please close this PR?

Thanks!

Shwetajain148 commented 6 years ago

Since this has already been handled, closing this one.