Merge pull request #249 from ctimmerm/ie8-onreadystatechange

Fixing IE8 support by bringing back onreadystatechange

lib/adapters/xhr.js

@@ -16,11 +16,15 @@ module.exports = function xhrAdapter(resolve, reject, config) {
   }
 
   var request = new XMLHttpRequest();
+  var loadEvent = 'onreadystatechange';
+  var xDomain = false;
 
   // For IE 8/9 CORS support
   // Only supports POST and GET calls and doesn't returns the response headers.
   if (window.XDomainRequest && !('withCredentials' in request) && !isURLSameOrigin(config.url)) {
     request = new window.XDomainRequest();
+    loadEvent = 'onload';
+    xDomain = true;
   }
 
   // HTTP basic authentication
@@ -36,13 +40,20 @@ module.exports = function xhrAdapter(resolve, reject, config) {
   request.timeout = config.timeout;
 
   // Listen for ready state
-  request.onload = function handleLoad() {
-    if (!request) {
+  request[loadEvent] = function handleLoad() {
+    if (!request || (request.readyState !== 4 && !xDomain)) {
       return;
     }
+
+    // The request errored out and we didn't get a response, this will be
+    // handled by onerror instead
+    if (request.status === 0) {
+      return;
+    }
+
     // Prepare the response
     var responseHeaders = 'getAllResponseHeaders' in request ? parseHeaders(request.getAllResponseHeaders()) : null;
-    var responseData = ['text', ''].indexOf(config.responseType || '') !== -1 ? request.responseText : request.response;
+    var responseData = !config.responseType || config.responseType === 'text' ? request.responseText : request.response;
     var response = {
       data: transformData(
         responseData,