Closed Bug 812130 Opened 8 years ago Closed 8 years ago

Increase the account provisioner timeout to greater than the server timeout

Categories

(Thunderbird :: Account Manager, defect)

defect
Not set
normal

Tracking

(thunderbird18+ fixed, thunderbird-esr1718+ fixed)

RESOLVED FIXED
Thunderbird 19.0
Tracking Status
thunderbird18 + fixed
thunderbird-esr17 18+ fixed

People

(Reporter: standard8, Assigned: bwinton)

References

Details

Attachments

(1 file)

The account provisioner timeout on individual providers is 10 seconds (run in async mode), but the server timeout is also 10 seconds.

We should increase Thunderbird's timeout slightly, to say 11 seconds, or maybe just 15 to ensure that even in a timeout situation, Thunderbird will still receive and display results for providers that haven't timed out (as realised in bug 812101).
Untested, but simple enough that I'm quite confident.
Assignee: nobody → bwinton
Status: NEW → ASSIGNED
Attachment #682015 - Flags: review?(mconley)
Comment on attachment 682015 [details] [diff] [review]
Change the timeout to 15 seconds.

Review of attachment 682015 [details] [diff] [review]:
-----------------------------------------------------------------

This is correct. Thanks Blake!
Attachment #682015 - Flags: review?(mconley) → review+
Landed on comm-central as https://hg.mozilla.org/comm-central/rev/a82b990ea07f
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 19.0
Comment on attachment 682015 [details] [diff] [review]
Change the timeout to 15 seconds.

[Triage Comment]
Lets get this on aurora as well, as I'd like to make it safer for the next releases as well (and we'll get onto esr once that is up and running).
Attachment #682015 - Flags: approval-comm-aurora+
Comment on attachment 682015 [details] [diff] [review]
Change the timeout to 15 seconds.

Landed

https://hg.mozilla.org/releases/comm-aurora/rev/367783b7c4a1

Also flagging for next release.
Attachment #682015 - Flags: approval-comm-esr17?
Attachment #682015 - Flags: approval-comm-esr17? → approval-comm-esr17+
You need to log in before you can comment on or make changes to this bug.