[Email] Improve IMAP retry logic

RESOLVED FIXED in Firefox OS v2.0

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: mcav, Assigned: mcav)

Tracking

unspecified
2.1 S1 (1aug)
x86_64
Linux
Dependency tree / graph

Firefox Tracking Flags

(blocking-b2g:2.0+, b2g-v2.0 fixed, b2g-v2.1 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
As discussed with :asuth re bug 1018828; make IMAP succeed even when given a zombie connection.
(Assignee)

Updated

4 years ago
Assignee: nobody → m
(Assignee)

Updated

4 years ago
Blocks: 1018828
(Assignee)

Updated

4 years ago
Attachment #8460514 - Flags: review?(bugmail)
Note for triage: this is a dependent bug of the formerly blocking bug 1018828. We converted that one to a meta bug and the actual work will happen here and the in the sibling bugs. Requesting blocking here for improvements to email sync stability in 2.0.
blocking-b2g: --- → 2.0?
blocking-b2g: 2.0? → 2.0+
(Assignee)

Updated

4 years ago
Attachment #8460514 - Flags: review?(bugmail)
(Assignee)

Comment 3

4 years ago
Comment on attachment 8460514 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia-email-libs-and-more/pull/324

Okay, I think this test is correct now. It tests that (A) the retry succeeds, and (B) the retry fails (and that it doesn't attempt another retry), and that there are no precommands left at the end.
Attachment #8460514 - Flags: review?(bugmail)
Blocks: 1044179
Comment on attachment 8460514 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia-email-libs-and-more/pull/324

Looks great!
Attachment #8460514 - Flags: review?(bugmail) → review+
(Assignee)

Updated

4 years ago
status-b2g-v2.0: --- → affected
status-b2g-v2.1: --- → fixed
Look forward to testing this one. :)
v2.0: https://github.com/mozilla-b2g/gaia/commit/99ac78acfe0b56b9eaf5ef3266ddbc2171c0c471
status-b2g-v2.0: affected → fixed
Target Milestone: --- → 2.1 S1 (1aug)
You need to log in before you can comment on or make changes to this bug.