Chromium Code Reviews Chromium Code Reviews
Issue 1114543002:
UMA to track the reason for re-auth (draft). (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: tools/metrics/histograms/histograms.xml |
| diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml |
| index 37cf72821846b039be376792c1a6e894e36f16c1..8983d4e735c2939bff33cf8ed188c24390bb043f 100644 |
| --- a/tools/metrics/histograms/histograms.xml |
| +++ b/tools/metrics/histograms/histograms.xml |
| @@ -13761,6 +13761,13 @@ Therefore, the affected-histogram name has to have at least one dot in it. |
| </summary> |
| </histogram> |
| +<histogram name="Login.ReauthReason" enum="LoginReauthReasons"> |
| + <owner>ginkage@chromium.org</owner> |
| + <summary> |
| + Tracks the reason why a user was sent through the re-auth flow. |
|
Ilya Sherman
2015/04/28 20:07:18
It's not clear to me from this description when th
Nikita (slow)
2015/04/29 09:21:46
nit: GAIA re-auth flow
Ivan Podogov
2015/04/29 10:34:25
This should not be hit, this value is used to indi
Ivan Podogov
2015/04/29 10:34:25
Done.
|
| + </summary> |
| +</histogram> |
| + |
| <histogram name="Login.StateKeyGenerationStatus" |
| enum="LoginStateKeyGenerationStatus"> |
| <owner>mnissler@chromium.org</owner> |
| @@ -55685,6 +55692,40 @@ To add a new entry, add it with any value and run test to compute valid value. |
| <int value="43" label="RESERVED">Reserved</int> |
| </enum> |
| +<enum name="LoginReauthReasons" type="int"> |
| + <int value="0" label="None"> |
| + No reason recorded so far, nothing to report. |
| + </int> |
| + <int value="1" label="InvalidTokenHandle"> |
| + Password changed, revoked credentials, account deleted. |
| + </int> |
| + <int value="2" label="IncorrectPwdEntered"> |
| + Incorrect password entered 3 times at the pod. |
| + </int> |
| + <int value="3" label="IncorrectSAMLPwdEntered"> |
| + Incorrect password entered by a SAML user once. OS would show a tooltip |
| + offering user to complete the online sign-in. |
| + </int> |
| + <int value="4" label="SAMLReauthPolicy"> |
| + Company policy required re-auth for SAML users. |
| + </int> |
| + <int value="5" label="MissingCryptohome"> |
| + Cryptohome is missing, most likely due to deletion during garbage |
| + collection. |
| + </int> |
| + <int value="6" label="SyncFailed"> |
| + After prior login, OS failed to connect to the sync with the existing RT. |
| + This could be due to a deleted account, password changed, account revoked, |
| + etc. |
| + </int> |
| + <int value="7" label="PwdUpdateSkipped"> |
| + User cancelled the password change prompt. Chrome OS has to continue to send |
| + the user through the online flow until user updates their cryptohome |
| + password or agrees to start the new cryptohome. |
| + </int> |
| + <int value="8" label="Other">Legacy profile holders.</int> |
| +</enum> |
| + |
| <enum name="LoginStateKeyGenerationStatus" type="int"> |
| <summary>The result of a state key generation operation.</summary> |
| <int value="0" label="GENERATION_METHOD_IDENTIFIER_HASH"> |
