-
Notifications
You must be signed in to change notification settings - Fork 151
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
Feature proposal: sync-xhr #126
Comments
Anything left before calling this done? |
Not for this issue, I suppose -- the PR is still out on whatwg/xhr#178, waiting on the changes to be made to HTML before it lands, I believe. |
I think I was happy to land whatwg/xhr#177 provided implementation bugs were filed, but I see that in whatwg/xhr#177 you did note it depended on changes to HTML. |
This has landed in the spec and shipped in Chrome. Can we just add it to features.md and mark this as closed please? BTW is anything else missing from features.md? |
Added. features.md could use a few changes -- It uses a normative voice when it really shouldn't, and should probably, in most cases, just link directly to the relevant specs, now that those are starting to integrate with FP themselves. Eventually all of that documentation should be moved somewhere like MDN, but features.md may need to stick around for a while to document the things that aren't really standards yet. |
Posting this here for completeness, there is a corresponding issue at whatwg/xhr#178 for the XHR spec change.
The proposal is to add a new feature,
sync-xhr
, which is enabled by default (default allowlist is*
)When disabled, calling
send()
on an XMLHttpRequest object with the async flag set to false will cause a NetworkError exception to be thrown. (This is chosen to avoid adding a new failure path for existing code which makes use of synchronous XHR)The text was updated successfully, but these errors were encountered: