Reputation: 1052
This question seems longer than it actually is, but I tried to keep a clean structure.
To start off, I have this webapp which needs to be very small in filesize to be able to load completely (without cache) on your smartphone in < 10s. This requires me to not use a library like jQuery (which doesn't matter anyway, see below).
I basically used the MDN Ajax page to create this short wrapper:
function ajax(options) {
var request;
if (window.XMLHttpRequest) {
request = new XMLHttpRequest();
} else {
request = new ActiveXObject("Microsoft.XMLHTTP");
}
request.timeout = 6500;
request.ontimeout = function () {
// not called, 6.5s is too high for a timeout anyway, checked this one first
options.error(request);
};
request.onreadystatechange = function () {
if (request.readyState == 4) {
if (request.status >= 200 && request.status < 400) {
options.success(request.responseText);
} else {
options.error(request);
}
}
};
request.open("GET", options.url, true);
request.send();
}
However, sometimes (in like 1/15 up to to 1/100 cases) the request fails. I get a request object similar to this one:
XMLHttpRequest {
onreadystatechange: ajax/request.onreadystatechange(),
readyState: 4,
timeout: 6500,
withCredentials: false,
upload: XMLHttpRequestUpload,
responseURL: "my-url",
status: 0, // <-- this seems to be the main problem
statusText: "",
responseType: "",
response: ""
}
To point this out, I'm not trying to do CORS here, it's all on the same domain. Also the timeout is set to 6.5s, if I look at the other requests, they usually complete within ~45ms. I tried to "retry" the Ajax 3x if it fails once, to catch the "status = 0"-problem, however this didn't always work out yet as it still occured and 0 means the webserver didn't deliver any content (this workaround just seems like a really dirty hack).
On another project I'm using jQuery because I don't have these size limits. I'm using a regular jQuery.ajax call, just like this:
$.ajax({
url: url,
dataType: "json",
success: options.updateSuccess,
error: options.updateError,
timeout: options.updateTimeout
});
However even this one sometimes fails with statusCode = 0 (the updateTimeout
is set to 65s as I "can wait for this one"). Also it's a different webserver than in the first snippet.
"&cache=" + new Date().getTime()
, didn't work too)Why does this happen then? I'm not capable of reproducing it repeatedly, it just sometimes happens. Is this a browser bug that makes every n-th Ajax call fail? Do you have any different approaches to solve this (preferably with plain Javascript)?
I did some more research, this time with wireshark. I logged which request caused the status = 0
and then viewed the TCP flow graph of these packages and all requests got an answer from the server, I guess it's not a server side problem.
Upvotes: 4
Views: 3423
Reputation: 1052
So I resolved both problems by doing a lot of tests. The jQuery one was quite easy, updating to jQuery 2.1.4 solved it, I don't want to know what was causing this, it just works now.
For the second problem I reduced the timeout to 500ms and then ran into a timeout which led me to onreadystate
being called even though the request didn't finish (in terms of getting an answer in time) completely thus getting status = 0
. Why this happened before, see above. To make sure that the timeout was not just "random" I added a retry counter and modified my Ajax function:
function ajax(options) {
var request;
if (options.retry === undefined) {
options.retry = 3; // modify here for default retry count
}
if (window.XMLHttpRequest) {
request = new XMLHttpRequest();
} else {
request = new ActiveXObject("Microsoft.XMLHTTP");
}
request.timeout = options.timeout ? options.timeout : 500; // modify here for default timeout
request.ontimeout = function () {
if (options.retry > 0) {
options.retry--;
ajax(options);
} else {
options.error(request);
}
};
request.onreadystatechange = function () {
var delta = new Date().getTime() - request.startTime;
if (request.readyState == 4) {
if (request.status >= 200 && request.status < 400) {
options.success(request.responseText);
} else if (request.status === 0 && delta >= request.timeout) {
// do nothing as this was "just a timeout"
} else {
options.error(request);
}
}
};
request.startTime = new Date().getTime();
request.open("GET", options.url, true);
request.send();
}
Upvotes: 3