If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

[Email] Improve IMAP retry logic

RESOLVED FIXED in Firefox OS v2.0

Status

Firefox OS
Gaia::E-Mail
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: mcav, Assigned: mcav)

Tracking

(Blocks: 1 bug)

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

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

Updated

3 years ago
Assignee: nobody → m
(Assignee)

Updated

3 years ago
Blocks: 1018828
(Assignee)

Comment 1

3 years ago
Created attachment 8460514 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia-email-libs-and-more/pull/324
(Assignee)

Updated

3 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

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

Comment 3

3 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)

Comment 5

3 years ago
Gaia master: https://github.com/mozilla-b2g/gaia/commit/bd09b12126c8d39027c82168ed8b9c843e168611

gelam: https://github.com/mozilla-b2g/gaia-email-libs-and-more/commit/8ece6e3c76ace7d101aa00359fa66efeaa5178c3
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Assignee)

Updated

3 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.