#8135 closed bug (fixed)
ajax connection error not correctly handled in Firefox
| Reported by: | john.firebaugh | Owned by: | jaubourg |
|---|---|---|---|
| Priority: | high | Milestone: | 1.5.1 |
| Component: | ajax | Version: | 1.5 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
Configure a server to serve the following static page, and visit it in Firefox:
https://gist.github.com/807090
Stop the server, then click the button.
Expected result: error alert.
Actual result: no alert is shown, and the following error is printed in the Firebug console:
uncaught exception: [Exception... "Component returned failure code: 0x80040111 (NS_ERROR_NOT_AVAILABLE) [nsIXMLHttpRequest.getAllResponseHeaders]" nsresult: "0x80040111 (NS_ERROR_NOT_AVAILABLE)" location: "JS frame :: http://0.0.0.0:3000/javascripts/jquery-1.5.js :: anonymous :: line 7207" data: no]
Change History (7)
comment:1 Changed 13 years ago by
comment:2 Changed 13 years ago by
| Cc: | jaubourg added |
|---|---|
| Component: | unfiled → ajax |
| Priority: | undecided → high |
| Status: | new → open |
http://bit.ly/e9l2Rz <-- description
FF bug ticket <-- unfixed since 2004(!!)
comment:3 Changed 13 years ago by
| Owner: | set to jaubourg |
|---|---|
| Status: | open → assigned |
comment:4 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Fixes #8135. Makes sure any exception thrown by Firefox when trying to access an XMLHttpRequest property when a network error occured is caught and notified as an error. Added test/networkerror.html to test the behavior.
Changeset: b90369e8cb2b6f3cc0afa34d815958ff0b605874
comment:5 Changed 13 years ago by
do you think it would make sense to make it a general try/catch block rather than specific to FireFox? Like the try/catch for Firefox above around setting the xhr.setRequestHeader?
But *THIS* patch is fantastic! Thank you *very* much for doing it.
comment:6 Changed 13 years ago by
| Cc: | jaubourg removed |
|---|---|
| Milestone: | 1.next → 1.5.1 |
Firefox version 3.6.13.