Closed Bug 549457 Opened 10 years ago Closed 10 years ago
errors doing STARTTLS ignored
We're not telling the user when doing an imap connection with STARTTLS fails. This can happen if the server advertises STARTTLS, but we fail to negotiate a connection, and the server resets the connection. I have a patch for this. Not sure I can write a unit test for this, but I may be able to trick fakeserver into it. The proposed fix makes sure we pass along errors when we're not retrying a url, and makes sure we retry urls if we fail during the logon process.
This tweaks the retry url logic so that we get told about the server resetting the connection on the second attempt, and extends the fakeserver to optionally simulate this behavior on starttls, and adds a unit test that verifies that we get an alert.
This is one of several fixes for our handling of connection problems that I'd really like to get into 3.1 beta 2 for more testing.
Status: NEW → ASSIGNED
Comment on attachment 429878 [details] [diff] [review] fix with unit test There's a little bit of bitrot, but I fixed that and tried running the tests, when I did, I got: TEST-UNEXPECTED-FAIL | (xpcshell/head.js) | ReferenceError: run_test is not defined - See following stack: JS frame :: /Users/moztest/comm/main/src/mozilla/testing/xpcshell/head.js :: _execute_test :: line 151 /Users/moztest/comm/main/src/mozilla/testing/xpcshell/head.js:129: TypeError: parts is null From the looks of it, it couldn't get the js stack for some reason.
this test still passes for me, once I fixed the bitrot. I'll try it on a mac build next.
oops, wrong patch
Attachment #434634 - Attachment is obsolete: true
Comment on attachment 434648 [details] [diff] [review] bit-rot fix with fake server changes sorry to do this to you, but could you retry this? It works fine for me on the mac as well.
Comment on attachment 434648 [details] [diff] [review] bit-rot fix with fake server changes Yep, that's working better.
fixed for trunk.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: [has patch for review standard8]
You need to log in before you can comment on or make changes to this bug.