Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

fix($httpBackend): Fix for IE9 AJAX request aborting error. #4529

Closed
wants to merge 5 commits into from

Conversation

lucipacurar
Copy link

Aborting AJAX requests in IE9 makes the XHR object's properties inaccessible and throws the following error:

SCRIPT575: Could not complete the operation due to error c00c023f.

@mary-poppins
Copy link

Thanks for the PR!

  • Contributor signed CLA now or in the past
    • If you just signed, leave a comment here with your real name
  • PR's commit messages follow the commit message format

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

@lucipacurar
Copy link
Author

Running "jasmine_node" task
Warning: Cannot find module 'mkdirp'� Use --force to continue.
Aborted due to warnings.
    >> 
    Warning: Task "parallel:travis" failed.� Use --force to continue.
Aborted due to warnings.

Did I break this?

@gchurchley
Copy link

I am seeing a problem that appears to be the same issue. See Issue #4585

I have created a single html page that runs on IE10, Chrome, and Firefox, and clearly shows the error on IE9:
http://www.apxproto.com/api/ngPost.htm

Perhaps you could use that page for testing the fix?

@lucipacurar
Copy link
Author

The fix works just fine on my local setup. As far as I can see the Travis CI tests aren't failing, it looks like a nodejs issue.

@lucipacurar
Copy link
Author

Can somebody review this commit? This is an important issue for people using AngularJS on IE.

LE: Pull request #4940 fixes the same issue. I believe my PR can be closed then.

@petebacondarwin
Copy link
Contributor

@lucassp - thanks for the PR. The failing tests were just travis being flaky. I agree though that this is superceded by #4940.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants