Closed
Bug 658221
Opened 13 years ago
Closed 11 years ago
Add a timeout to ajax request in TBPL
Categories
(Tree Management Graveyard :: TBPL, defect)
Tree Management Graveyard
TBPL
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: mounir, Assigned: mounir)
References
Details
Attachments
(1 file)
2.11 KB,
patch
|
mstange
:
review+
|
Details | Diff | Splinter Review |
Because of bug 657889, we can't access to build.mozilla.org from the Paris office which makes TBPL always showing "Loading 50%" with no error message. This patch adds a timeout value (10s) so an error message is shown when the URL isn't accessible.
Attachment #533584 -
Flags: review?
Assignee | ||
Updated•13 years ago
|
Attachment #533584 -
Flags: review? → review?(mstange)
Assignee | ||
Updated•13 years ago
|
Whiteboard: [needs review]
Updated•13 years ago
|
Attachment #533584 -
Flags: review?(mstange) → review+
Assignee | ||
Comment 1•13 years ago
|
||
Pushed: https://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/8c1153a543a5
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [needs review]
Comment 2•13 years ago
|
||
Ah, this was why I was claiming in bug 657540 comment 6 that tip was broken, and would only load pushes and pending but not tinderbox, or pushes and tinderbox but not pending. I'm real happy that you all have nice fast connections, but if tbpl.m.o is going to insist that every part must load in 10 seconds or less, even when I'm on my fringe 3G connection restoring a session with three trees open, then I won't be able to use it anymore.
Comment 3•13 years ago
|
||
Can we add a query param for the timeout so it's possible to override the default value?
Comment 4•13 years ago
|
||
And apparently we only show the error message for a few parts, since I've only seen it once in 30 or 40 failed broken partial loads.
Comment 5•13 years ago
|
||
Backed out: http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/a1838a15ad48 http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/512e82d808ca
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 6•11 years ago
|
||
Marking this as WONTFIX given that I'm not planning to fix this. Feel free to reopen if someone wants to fix this bug.
Status: REOPENED → RESOLVED
Closed: 13 years ago → 11 years ago
Resolution: --- → WONTFIX
Updated•10 years ago
|
Product: Webtools → Tree Management
Updated•9 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•