This repository was archived by the owner on Apr 12, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 27.4k
Uncaught exception: CORS failures with responseType "document" #1922
Labels
Comments
Turns out this is from accessing responseText after a failed request. Might be a Chrome bug. |
Okay, so XHR2 says this about responseText: If responseType is not the empty string or "text", throw an "InvalidStateError" exception and terminate these steps. So we should not access reponseText in this case. |
Anyone know why we query responseText at all? Is it a compat thing? |
@taralx you are right: http://www.w3.org/TR/XMLHttpRequest/#dom-xmlhttprequest-responsetext the responseText check is redundant and should be removed. some tests where we mock responseText will need to be changed as well. |
gonzaloruizdevilla
added a commit
to gonzaloruizdevilla/angular.js
that referenced
this issue
Feb 22, 2013
fixes angular#1922 Signed-off-by: Gonzalo Ruiz de Villa <gonzalo.ruizdevilla@adesis.com>
IgorMinar
pushed a commit
to IgorMinar/angular.js
that referenced
this issue
Feb 24, 2013
…ular#1922 Signed-off-by: Gonzalo Ruiz de Villa <gonzalo.ruizdevilla@adesis.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Testcase: http://taral.dreamhosters.com/eve-test/corsreject.html
Expected result "Result: FAILURE"
Actual result "Result: " and an uncaught exception in the console.
Browser: Google Chrome 24.0.1312.56 and 26.0.1398.0
The text was updated successfully, but these errors were encountered: