Closed
Bug 985504
Opened 11 years ago
Closed 4 years ago
Solicit user credentials once on receiving invalid-client-state response from server
Categories
(Firefox for Android Graveyard :: Android Sync, defect, P5)
Tracking
(fennec+)
RESOLVED
INCOMPLETE
Tracking | Status | |
---|---|---|
fennec | + | --- |
People
(Reporter: rnewman, Assigned: nalexander)
References
Details
(Whiteboard: [qa+])
AaronMT just saw an odd situation in which both desktop and Android starting receiving invalid-client-state errors from the server.
This might well be a server bug, but still, there are things we should do better:
* On receiving invalid-client-state, either immediately or after a number of retries, transition to Separated, not Cohabiting.
* Ask the user for credentials, persisting a derivative of the old credentials to determine whether the user re-entered the old ones. If they entered the old ones again, there's no point continuing to ask; enter the Android account hard error state and be done.
There's a balance here between detecting a "real" problem -- perhaps the user's key changed elsewhere, and i-c-s is accurate -- and being robust against some kind of transient error.
Comment 1•11 years ago
|
||
Comment 2•11 years ago
|
||
are there STR? Train-06, going out this afternoon, fixes a bug in js-client that has this same error.
Updated•11 years ago
|
Whiteboard: [qa+]
Comment 3•11 years ago
|
||
Removed, re-added desktop account, still stuck in a re-auth loop:
https://gist.github.com/AaronMT/7e8ed47d96736593a6e0
Comment 4•11 years ago
|
||
i suspect you're seeing this: https://bugzilla.mozilla.org/show_bug.cgi?id=982798
train6 should go out later today.
Comment 5•11 years ago
|
||
No password change attempted.
Comment 6•11 years ago
|
||
Per Bug 985611, it looks like something made a request to this account with an empty X-Client-State header. This "should never happen" but clearly it has. I'll file some bugs about server-side mitigation to prevent this in future.
You should be able to get the account out of this stage by resetting your password. We'll work on a slightly less brutal recovery path.
Comment 7•11 years ago
|
||
(In reply to Ryan Kelly [:rfkelly] from comment #6)
> You should be able to get the account out of this stage by resetting your
> password. We'll work on a slightly less brutal recovery path.
Confirmed working now by doing so.
Updated•11 years ago
|
tracking-fennec: --- → ?
Updated•11 years ago
|
Assignee: nobody → nalexander
tracking-fennec: ? → 31+
Reporter | ||
Comment 8•11 years ago
|
||
Being realistic about tracking, because we have lots and lots of things to do.
Do we still think this is a problem in the wild, chaps?
tracking-fennec: 31+ → +
Updated•7 years ago
|
Product: Android Background Services → Firefox for Android
Comment 10•4 years ago
|
||
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
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
•