From e314ab00b48be01283f4c48fc2ecaef73fac7c85 Mon Sep 17 00:00:00 2001 From: Rikki Gibson Date: Sat, 28 Jul 2018 09:46:33 -0700 Subject: [PATCH] Remove HTTP 1223 handling The 1223 error was fixed in IE 10. See https://blogs.msdn.microsoft.com/ieinternals/2012/03/01/internet-explorer-10-consumer-preview-minor-changes-list/ --- lib/adapters/xhr.js | 5 ++--- test/specs/requests.spec.js | 22 ---------------------- 2 files changed, 2 insertions(+), 25 deletions(-) diff --git a/lib/adapters/xhr.js b/lib/adapters/xhr.js index e1aba37c9b..239ccfc52c 100644 --- a/lib/adapters/xhr.js +++ b/lib/adapters/xhr.js @@ -49,9 +49,8 @@ module.exports = function xhrAdapter(config) { var responseData = !config.responseType || config.responseType === 'text' ? request.responseText : request.response; var response = { data: responseData, - // IE sends 1223 instead of 204 (https://github.com/axios/axios/issues/201) - status: request.status === 1223 ? 204 : request.status, - statusText: request.status === 1223 ? 'No Content' : request.statusText, + status: request.status, + statusText: request.statusText, headers: responseHeaders, config: config, request: request diff --git a/test/specs/requests.spec.js b/test/specs/requests.spec.js index 0a272b67ad..f823453e45 100644 --- a/test/specs/requests.spec.js +++ b/test/specs/requests.spec.js @@ -220,28 +220,6 @@ describe('requests', function () { }); }); - // https://github.com/axios/axios/issues/201 - it('should fix IE no content error', function (done) { - var response; - - axios('/foo').then(function (res) { - response = res - }); - - getAjaxRequest().then(function (request) { - request.respondWith({ - status: 1223, - statusText: 'Unknown' - }); - - setTimeout(function () { - expect(response.status).toEqual(204); - expect(response.statusText).toEqual('No Content'); - done(); - }, 100); - }); - }); - it('should allow overriding Content-Type header case-insensitive', function (done) { var response; var contentType = 'application/vnd.myapp.type+json';