-
Notifications
You must be signed in to change notification settings - Fork 2.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test Atomic HTTP redirect handling #55
Conversation
Gah, PhantomJS doesn't implement 308. |
Section 4.2 steps 4.4 and 4.10 say yes. |
@josh XMLHttpRequest has |
Nice! but looks like Safari still doesn't. I doubt IE does either. I wonder what we should set In a pre-responseURL world, github.com sends back a
|
All tests pass on Chrome Canary (native fetch), Chrome and Firefox. Currently failing on Safari and IE because of missing |
I'm hesitant to move some of our github.com header logic into the polyfill. The native fetch implementations won't have that header, so polyfill users could become dependent on non-standard behavior that disappears when browsers enable |
Native implementations already do the right thing. The header does nothing in that case. |
Test Atomic HTTP redirect handling
https://fetch.spec.whatwg.org/#atomic-http-redirect-handling
Should behave just like XHR's redirection.
@annevk is
Response.prototype.url
supposed to be the redirected location? That was one of my feature wish list items that XHR can't do.Closes #42.
/cc @dgraham @annevk