From 97cbfbeddaa6f02fa89d14e5e246db6708587b72 Mon Sep 17 00:00:00 2001 From: Mattias Hellborg Arthursson Date: Wed, 14 Aug 2013 14:29:22 +0200 Subject: [PATCH] LDAP-230: Pool Validation polluting logs with "IOException: connection closed" stacktrace even though it is handled gracefully by framework Lowered to log level to debug. --- .../validation/DefaultDirContextValidator.java | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/core/src/main/java/org/springframework/ldap/pool/validation/DefaultDirContextValidator.java b/core/src/main/java/org/springframework/ldap/pool/validation/DefaultDirContextValidator.java index 383e4240..2a6eeb8f 100644 --- a/core/src/main/java/org/springframework/ldap/pool/validation/DefaultDirContextValidator.java +++ b/core/src/main/java/org/springframework/ldap/pool/validation/DefaultDirContextValidator.java @@ -16,15 +16,15 @@ package org.springframework.ldap.pool.validation; -import javax.naming.NamingEnumeration; -import javax.naming.directory.DirContext; -import javax.naming.directory.SearchControls; - import org.apache.commons.lang.Validate; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.springframework.ldap.pool.DirContextType; +import javax.naming.NamingEnumeration; +import javax.naming.directory.DirContext; +import javax.naming.directory.SearchControls; + /** * Default {@link DirContext} validator that executes {@link DirContext#search(String, String, SearchControls)}. The * name, filter and {@link SearchControls} are all configurable. There is no special handling for read only versus @@ -174,11 +174,13 @@ public class DefaultDirContextValidator implements DirContextValidator { } } catch (Exception e) { - this.logger.warn("DirContext '" + dirContext + "' failed validation with an exception.", e); + if(this.logger.isDebugEnabled()) { + this.logger.debug("DirContext '" + dirContext + "' failed validation with an exception.", e); + } } - if (this.logger.isInfoEnabled()) { - this.logger.info("DirContext '" + dirContext + "' failed validation."); + if (this.logger.isDebugEnabled()) { + this.logger.debug("DirContext '" + dirContext + "' failed validation."); } return false; }