Closed Bug 987853 Opened 11 years ago Closed 11 years ago

Loop Server — Add a "retry after" header when doing 503s

Categories

(Hello (Loop) :: Server, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: alexis+bugs, Assigned: rhubscher)

References

Details

Attachments

(1 file)

That's something we've been doing for Sync, that could be useful for us here. When sent together with an HTTP 503 status code, this header signifies that the server is undergoing maintenance. The client should not attempt any further requests to the server for the number of seconds specified in the header value.
For this bug, I am going to add a value from the configuration file (30s)
Assignee: nobody → rhubscher
Attached file Link to GitHub PR.
Attachment #8407385 - Flags: review?(alexis+bugs)
Status: NEW → ASSIGNED
Comment on attachment 8407385 [details] [review] Link to GitHub PR. Taking over the review, LGTM
Attachment #8407385 - Flags: review?(alexis+bugs) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Verified in code. We shall see how well Stage handles 503s :-)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: