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

Changed status to explicitly check for status 0

  • Loading branch information
Clark, Jason Clark, Jason
Clark, Jason authored and Clark, Jason committed Aug 3, 2016
commit 41a098fa393be5e2dc75b751fb30fbc801676e84

Some generated files are not rendered by default. Learn more.

Some generated files are not rendered by default. Learn more.

@@ -168,7 +168,7 @@
xmlHttp.setRequestHeader('Content-Type', 'text/plain');
xmlHttp.send(JSON.stringify(data));
xmlHttp.onreadystatechange = function() {
if(xmlHttp.readyState === 4 && xmlHttp.status !== 200) {
if(xmlHttp.readyState === 4 && xmlHttp.status === 0) {
if(attemptCounter <= 12) {
console.log("Failed to log to loggly because of a network condition.");
console.log("Will attempt to log again in 5 seconds.");
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.