summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Pursehouse <david.pursehouse@sonymobile.com>2016-02-03 15:25:50 +0900
committerDavid Pursehouse <david.pursehouse@sonymobile.com>2016-02-03 18:27:18 +0900
commit7f64ab66b89c861bb744d513dfb50ee574bb7d9e (patch)
treed6c6dd1c408d14746848ef2d2a09a54ccf731814
parenta2b943bff271f93e3225a40796202994b06801dc (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.java4
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();
}
}