Closed
Bug 1224203
Opened 9 years ago
Closed 9 years ago
Detecting Server Unreachable takes several minutes
Categories
(Firefox OS Graveyard :: Sync, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
2.6 S2 - 12/4
People
(Reporter: mbdejong, Assigned: mbdejong)
References
Details
Attachments
(3 files)
No description provided.
|
Assignee | |
Comment 1•9 years ago
|
||
If you set '123.123.123.123 syncto.dev.mozaws.net' in /etc/hosts and click 'Sync Now' in B2G Sync settings, it takes about 4 minutes and 40 seconds before it times out. That's too long. :)
|
|
Assignee | |
Comment 2•9 years ago
|
||
Should test with https://github.com/Kinto/kinto.js/pull/264 - might well be the fix for this! :)
|
|
Assignee | |
Updated•9 years ago
|
Assignee: nobody → mbdejong
|
|
Assignee | |
Updated•9 years ago
|
Target Milestone: 2.6 S1 - 11/20 → 2.6 S2 - 12/4
|
||
Comment 3•9 years ago
|
||
|
|
Assignee | |
Updated•9 years ago
|
Attachment #8691994 -
Flags: review?(ferjmoreno)
|
||
Comment 4•9 years ago
|
||
Comment on attachment 8691994 [details] [review]
[gaia] michielbdejong:1224203-network-timeouts > mozilla-b2g:master
You need to rebase
Attachment #8691994 -
Flags: review?(ferjmoreno) → review+
|
|
||
Comment 6•9 years ago
|
||
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(ferjmoreno)
Resolution: --- → FIXED
|
|
||
Comment 7•9 years ago
|
||
I had to revert this commit cause the timeout policy was too strict
https://github.com/mozilla-b2g/gaia/commit/73ec13faf558bc2531a81ee2302f01a785795ef7
Status: RESOLVED → REOPENED
Flags: needinfo?(mbdejong)
Resolution: FIXED → ---
|
|
Assignee | |
Comment 8•9 years ago
|
||
The reason we had to revert this is that we left the timeout value at the 5 seconds default from Kinto.js, which is not realistic on mobile.
I will set the timeout to 180 seconds instead of 5 seconds, then it should be safe again.
Flags: needinfo?(mbdejong)
|
|
||
Comment 9•9 years ago
|
||
|
|
||
Comment 10•9 years ago
|
||
|
|
||
Comment 11•9 years ago
|
||
I've just sent an alternative PR to fix the linter error. Needing info me to remember to merge this tomorrow, once treeherder is done.
Flags: needinfo?(ferjmoreno)
|
|
||
Comment 12•9 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Flags: needinfo?(ferjmoreno)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•