Closed
Bug 969227
Opened 11 years ago
Closed 11 years ago
Handle X-Backoff headers
Categories
(Firefox for Android Graveyard :: Android Sync, defect)
Tracking
(firefox29 fixed, firefox30 fixed, fennec29+)
RESOLVED
FIXED
Firefox 30
People
(Reporter: rnewman, Assigned: rnewman)
References
Details
(Whiteboard: [qa-])
Attachments
(1 file)
57 bytes,
text/x-github-pull-request
|
rnewman
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Review |
Response headers from the storage and token servers are currently a heady mix of X-Weave* and X-*. We should handle both, just in case. https://docs.services.mozilla.com/token/apis.html#response-headers
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8374315 -
Flags: review?(nalexander)
Assignee | ||
Comment 2•11 years ago
|
||
Comment on attachment 8374315 [details] [review] Proposed patch. v1 [Approval Request Comment] Bug caused by (feature/regressing bug #): Hurried FxA development. User impact if declined: Partial Sync implementation. Testing completed (on m-c, etc.): Automated tests. Risk to taking this patch (and alternatives if risky): Failure to deliver expected level of quality for Fx29. String or IDL/UUID changes made by this patch: None. Nick reviewed on GitHub. Landed with tests in a-s. https://hg.mozilla.org/integration/fx-team/rev/e624202c4fdf
Attachment #8374315 -
Flags: review?(nalexander)
Attachment #8374315 -
Flags: review+
Attachment #8374315 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 3•11 years ago
|
||
This isn't something we can easily QA, so flagging qa- for now. If we have a chance to set up a staging token server and somehow point the client at it, then we can give it a shot, but this at least has automated tests.
Whiteboard: [qa-]
Target Milestone: --- → Firefox 30
Comment 4•11 years ago
|
||
Well, we do have this: https://token-stage3.stage.mozaws.net or some flavor of that depending on frequently OPs updates our TS Stage. Will that work?
Assignee | ||
Comment 5•11 years ago
|
||
(In reply to James Bonacci [:jbonacci] from comment #4) > Well, we do have this: https://token-stage3.stage.mozaws.net > or some flavor of that depending on frequently OPs updates our TS Stage. > Will that work? If you have the FxA and Sync servers to go with it, so we can do a build of Firefox pointing to those URLs. Might not be wasted effort, because we can use that setup to test Bug 958900, too.
Comment 6•11 years ago
|
||
FYI, I will wait for your tests before uplifting the patch.
Comment 7•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e624202c4fdf
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 8•11 years ago
|
||
Drat, I accidentally checked in some non-pngcrushed resources here. Will fix.
Assignee | ||
Comment 10•11 years ago
|
||
https://etherpad.mozilla.org/fxa-sync-backoff-test-plan
Assignee | ||
Comment 11•11 years ago
|
||
Relanded images: https://hg.mozilla.org/integration/fx-team/rev/d7f2934f2f29
Assignee | ||
Comment 12•11 years ago
|
||
This was tested with Services QA along with Bug 958900 and Bug 972613, Sylvestre. Please flag all three for uplift at your convenience.
Flags: needinfo?(sledru)
Updated•11 years ago
|
Attachment #8374315 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Flags: needinfo?(sledru)
Assignee | ||
Comment 13•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/a825e6857b1f
status-firefox29:
--- → fixed
status-firefox30:
--- → fixed
Updated•7 years ago
|
Product: Android Background Services → Firefox for Android
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•