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

Added gitignore and added retries for failed batch attempts #43

Open
wants to merge 8 commits into
base: master
from

Conversation

@riceboyler
Copy link

@riceboyler riceboyler commented Aug 3, 2016

To handle issues where there is lost internet connectivity, this will attempt to post logs for a minute, every 5 seconds. This is a first pass on it, in the future, configurable settings for attempts and interval could be added.

@riceboyler
Copy link
Author

@riceboyler riceboyler commented Aug 4, 2016

To view this PR without the whitespace changes (had to clean up the inconsistency, my OCD kicked in) you can go to https://github.com/loggly/loggly-jslogger/pull/43/files?w=1

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

1 participant
You can’t perform that action at this time.