2
0
mirror of https://github.com/tenrok/axios.git synced 2026-06-23 20:40:40 +03:00

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/
This commit is contained in:
Rikki Gibson
2018-07-28 09:46:33 -07:00
committed by Khaled Garbaya
parent 1c2881cbe6
commit a9831131c3
2 changed files with 2 additions and 25 deletions
+2 -3
View File
@@ -49,9 +49,8 @@ module.exports = function xhrAdapter(config) {
var responseData = !config.responseType || config.responseType === 'text' ? request.responseText : request.response; var responseData = !config.responseType || config.responseType === 'text' ? request.responseText : request.response;
var response = { var response = {
data: responseData, data: responseData,
// IE sends 1223 instead of 204 (https://github.com/axios/axios/issues/201) status: request.status,
status: request.status === 1223 ? 204 : request.status, statusText: request.statusText,
statusText: request.status === 1223 ? 'No Content' : request.statusText,
headers: responseHeaders, headers: responseHeaders,
config: config, config: config,
request: request request: request
-22
View File
@@ -245,28 +245,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) { it('should allow overriding Content-Type header case-insensitive', function (done) {
var response; var response;
var contentType = 'application/vnd.myapp.type+json'; var contentType = 'application/vnd.myapp.type+json';