developit / developit/unfetch

XDomainRequest. IE 8 will not work when CORS requests

Open
#106 6 comments 0 reactions 0 assignees View on GitHub
possible-addon
Dominant language
JavaScript
Stars
5.7k
Forks
204
PR merge metrics
No merged PRs in 30d

Description

## steps to reproduce

unfetch@4.0.1:

```html




fetch('page2.html')
.then(function(response) {
return response.text();
})
.then(function(data) {
console.log(data);
});

... any data ...
```

use any http server that supports Access-Control-Allow-Origin headers:

```bash
npm install http-server -g
http-server -p 8081 --cors
```
* --cors - to enable CORS via the Access-Control-Allow-Origin

### result

Through VM WinXP SP3 with **IE version: 8.0.6001.18702** Update version: 0

* F12 - Document Mode = 8 and 7.

and/or

* Win10 IE11 - F12 - Emulation - Document Mode = 7 and 8 / User agent: 7/8

**It does not work at all!**

## user-space solutions

Here's my example of hack for already released unfetch version like 4.0.1. Works fine for IE7+:

```javascript
var send = window.XMLHttpRequest.prototype.send;
window.XMLHttpRequest.prototype.send = function()
{
this.onreadystatechange = function()
{
// TODO: prevent duplicate calling of onload if it's not CORS!
if(this.readyState === 4 && this.status === 200) {
return this.onload.apply(this, arguments);
}
}
return send.apply(this, arguments);
}

// IE7+ ok now
```

Actually, even IE 11 with 5+ emulation, that of course strange :) because it should be ActiveXObject [before IE7](https://developer.mozilla.org/en-US/docs/Web/API/XMLHttpRequest/Using_XMLHttpRequest_in_IE6). Bad emulation :)

However, **for real IE8** CORS support we should have requests through **XDomainRequest** not for XMLHttpRequest. Thus, your `onload()` is silent.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with unfetch@4.0.1/polyfill/index.js and trace the XMLHttpRequest.prototype.send and onload handling described in the report. Reproduce the CORS request in IE8 or the documented IE7/8 emulation modes, then verify that the completed request invokes the expected callback through the appropriate legacy transport.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend, web-dev
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.