From 7f64ab66b89c861bb744d513dfb50ee574bb7d9e Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Wed, 3 Feb 2016 15:25:50 +0900 Subject: [PATCH] ldap.Helper: Kill "assuming empty group membership" logspam Change I75fd86fb9 added catching of AccountException and replaced the stack trace log with a warning: "Account 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 --- .../main/java/com/google/gerrit/server/auth/ldap/Helper.java | 4 ---- 1 file changed, 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(); } }