[signin] AccountReconcilor should logout less |
|||||||||||||||||||||
Issue descriptionIn some cases the AccountReconcilor does a full logout followed by a sequence of MergeSessions. This can be a problem if some error (invalid token, connectivity issue) happens during the reconcile after the logout has been done, as it leaves the users in a signed-out state. In particular at startup, when Chrome assumes that all invalid the tokens are valid again, the reconcile can fail after logout, and then eventually signin the user in the exact same state they were before the reconcile, which is unnecessary churn on the account. See design doc: https://docs.google.com/a/google.com/document/d/1GeqALUSPcUaisTnSTgNZEHxKXy7QOcPA_wk0gEu07fk/edit?usp=sharing And subsection in Dice doc: https://docs.google.com/document/d/1gx7_pjcWkCj52WABcAF_E1dsYnBJtwkxgsLvBooncqk/edit?ts=5967b289&pli=1#heading=h.o67k3qgm8vs4 ⛆ |
|
|
,
Oct 5 2017
For posterity, I started a CL to add support for oauth multilogin in google_apis: https://chromium-review.googlesource.com/c/chromium/src/+/702483 I'm not actively working on it, but it could be a good starting point when we pick up this bug again in the future.
,
Oct 5 2017
Could we include a metric for how many times we "unnecessarily" reconcile (and end up back in the same state)? Maybe a histogram which is a distribution over time for how long these unnecessary reconciles take?
,
Nov 6 2017
--Chrome Identity automated triaging-- This bug is Assigned and has gone one month without any activity, so it is being moved to Available to indicate that it is not actively being worked on. If you are working on this bug, please mark yourself as the owner and move back to Assigned. Please see https://goo.gl/78kbny for more details. Please remove the Services>SignIn or UI>Browser>Profiles components if this bug isn't related to Chrome Identity. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Apr 17 2018
Issue 501352 has been merged into this issue.
,
Aug 1
,
Aug 7
Issue 764305 has been merged into this issue.
,
Aug 7
Valeriya is going to work on this.
,
Aug 9
,
Aug 9
,
Aug 9
,
Aug 20
,
Aug 20
,
Aug 21
,
Aug 21
,
Aug 22
,
Sep 11
,
Sep 26
,
Sep 28
,
Oct 1
,
Oct 27
|
||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by droger@chromium.org
, Oct 5 2017