diff options
author | David Pursehouse <david.pursehouse@sonymobile.com> | 2016-02-03 15:25:50 +0900 |
---|---|---|
committer | David Pursehouse <david.pursehouse@sonymobile.com> | 2016-02-03 18:27:18 +0900 |
commit | 7f64ab66b89c861bb744d513dfb50ee574bb7d9e (patch) | |
tree | d6c6dd1c408d14746848ef2d2a09a54ccf731814 | |
parent | a2b943bff271f93e3225a40796202994b06801dc (diff) |
ldap.Helper: Kill "assuming empty group membership" logspam
Change I75fd86fb9 added catching of AccountException and replaced the
stack trace log with a warning:
"Account <name> not found: assuming empty group membership"
The intention was to prevent excessive logging, but these warnings
themselves are filling up the log.
Change-Id: Id52830dbcab1f7434960a47cb11e5e559a6f650a
-rw-r--r-- | gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/Helper.java | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/Helper.java b/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/Helper.java index 4c29c9b225..b246868314 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/Helper.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/auth/ldap/Helper.java @@ -224,8 +224,6 @@ import javax.security.auth.login.LoginException; try { account = findAccount(schema, ctx, username, false); } catch (AccountException e) { - LdapRealm.log.warn("Account " + username + - " not found, assuming empty group membership"); return Collections.emptySet(); } } @@ -247,8 +245,6 @@ import javax.security.auth.login.LoginException; try { account = findAccount(schema, ctx, username, true); } catch (AccountException e) { - LdapRealm.log.warn("Account " + username + - " not found, assuming empty group membership"); return Collections.emptySet(); } } |