diff options
| author | Lutz Justen <ljusten@chromium.org> | 2018-06-05 09:58:56 +0200 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-06-21 08:14:28 -0700 |
| commit | fdba47dba5c6f9560d88e22b1aed5c43715cbbc0 (patch) | |
| tree | 811174af3cf87dc39efaeaad476fb313525bf4be | |
| parent | 0e2a5fd43dc395b3fd526c8f726911871597dd82 (diff) | |
| download | system_api-fdba47dba5c6f9560d88e22b1aed5c43715cbbc0.tar.gz | |
authpolicy: Remove unused field
Removes last_auth_error from ActiveDirectoryUserStatus as it's not
neeeded anymore. It was supposed to detect whether the Active Directory
server can be accessed, but that turns out to be flaky since the
network might be switching during AuthenticateUser(). Calling
PingServer() in GetUserStatus() is more robust.
Also, Chrome never used it, so it's safe to remove.
CQ-DEPEND=CL:1090711
BUG=chromium:849318
TEST=trybots
Change-Id: I16d49d795becf813dab88357b3f611c3459cfbe4
Reviewed-on: https://chromium-review.googlesource.com/1086795
Commit-Ready: Lutz Justen <ljusten@chromium.org>
Tested-by: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Roman Sorokin <rsorokin@chromium.org>
| -rw-r--r-- | dbus/authpolicy/active_directory_info.proto | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/dbus/authpolicy/active_directory_info.proto b/dbus/authpolicy/active_directory_info.proto index 38c4fcb..aa85c12 100644 --- a/dbus/authpolicy/active_directory_info.proto +++ b/dbus/authpolicy/active_directory_info.proto @@ -158,8 +158,7 @@ message ActiveDirectoryUserStatus { optional TgtStatus tgt_status = 2; // Status of the user's password. optional PasswordStatus password_status = 3; - // Last error returned from AuthenticateUser D-Bus call. - optional ErrorType last_auth_error = 4; + reserved 4; // Next ID to use: 5 } |