Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update variable name #6

Merged
merged 1 commit into from Jan 20, 2017
Merged

Conversation

@Shwetajain148
Copy link

@Shwetajain148 Shwetajain148 commented Jan 20, 2017

No description provided.

@benjie
Copy link

@benjie benjie commented Jan 20, 2017

👍 Please let us know when a fix is released

@mchaudhary mchaudhary merged commit 0cd734c into loggly:master Jan 20, 2017
@mostlyjason
Copy link

@mostlyjason mostlyjason commented Jan 24, 2017

@benjie we have released a new beta version of this package with this fix along with some others to buffer messages and retry them automatically in the case of a connection error. Would you be willing to test it out?

npm install winston-loggly-bulk@beta

@benjie
Copy link

@benjie benjie commented Jan 25, 2017

👍 Am using that now, will let you know if I notice any issues.

@mostlyjason
Copy link

@mostlyjason mostlyjason commented Feb 2, 2017

@benjie Did you get a chance to try it yet? We\re planning to make it a production release soon and want to make sure there are no issues.

@benjie
Copy link

@benjie benjie commented Feb 2, 2017

Have been running it for the past 8 days and not noticed anything so I assume all is well 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.