Skip to content

Conversation

skrings
Copy link

@skrings skrings commented Jan 25, 2011

Hi,

this commits fixed an older Bug #5866 which appears since jQuery 1.4. In jQuery 1.4 the way we select the XMLHttpRequest or the ActiveXObject has changed. For preformance reasons the XMLHttpRequest is preferred if we can use it and thats okay. But we all know that Mircrosoft failed to implement the XMLHttpRequest in IE7.
So the XMLHttpRequest in IE7 aborts with an error if you fire an ajax request without protocol in url. The easiest way to get around is to add the protocol to the url if it´s missing.

I hope you push my little commit into the 1.5 branch.

Best regards,
Sören

jsFiddle: http://jsfiddle.net/WuGZh/
Bug: http://bugs.jquery.com/ticket/5866

@jaubourg
Copy link
Member

Why not a much simpler s.url.replace( rprotocol , protocol + "//" )?

@jaubourg
Copy link
Member

Fixed here: d7d6471
Replacement was put higher in ajax and provided some minor regexp and crossDomain detection optimizations.
Thanks a lot for the pull request, it was very valuable even if it didn't get pushed as is.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 20, 2019
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants