From 33840bc46cc302abef9531c5fca4b75ea09d6ea9 Mon Sep 17 00:00:00 2001 From: Andrei Ivanov Date: Tue, 11 Oct 2016 12:03:49 +0200 Subject: [PATCH] No need to log twice in case of an exception (#402) --- .../ldap/pool/validation/DefaultDirContextValidator.java | 1 + .../ldap/pool2/validation/DefaultDirContextValidator.java | 1 + 2 files changed, 2 insertions(+) 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 78491ecb..644cba66 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 @@ -181,6 +181,7 @@ public class DefaultDirContextValidator implements DirContextValidator { if(this.logger.isDebugEnabled()) { this.logger.debug("DirContext '" + dirContext + "' failed validation with an exception.", e); } + return false; } if (this.logger.isDebugEnabled()) { diff --git a/core/src/main/java/org/springframework/ldap/pool2/validation/DefaultDirContextValidator.java b/core/src/main/java/org/springframework/ldap/pool2/validation/DefaultDirContextValidator.java index 9b593032..ebccf61b 100644 --- a/core/src/main/java/org/springframework/ldap/pool2/validation/DefaultDirContextValidator.java +++ b/core/src/main/java/org/springframework/ldap/pool2/validation/DefaultDirContextValidator.java @@ -181,6 +181,7 @@ public class DefaultDirContextValidator implements DirContextValidator { if(this.logger.isDebugEnabled()) { this.logger.debug("DirContext '" + dirContext + "' failed validation with an exception.", e); } + return false; } if (this.logger.isDebugEnabled()) {