From c1739290022a4dbc369e067c1ea5a2c41df82f9e Mon Sep 17 00:00:00 2001 From: Michael Jumper Date: Fri, 23 Oct 2015 16:09:54 -0700 Subject: [PATCH] GUAC-1115: Log failures to bind with search DN. --- .../guacamole/auth/ldap/AuthenticationProviderService.java | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/extensions/guacamole-auth-ldap/src/main/java/org/glyptodon/guacamole/auth/ldap/AuthenticationProviderService.java b/extensions/guacamole-auth-ldap/src/main/java/org/glyptodon/guacamole/auth/ldap/AuthenticationProviderService.java index d10095bd2..24dd17f13 100644 --- a/extensions/guacamole-auth-ldap/src/main/java/org/glyptodon/guacamole/auth/ldap/AuthenticationProviderService.java +++ b/extensions/guacamole-auth-ldap/src/main/java/org/glyptodon/guacamole/auth/ldap/AuthenticationProviderService.java @@ -109,6 +109,12 @@ public class AuthenticationProviderService { confService.getSearchBindPassword() ); + // Warn of failure to find + if (searchConnection == null) { + logger.error("Unable to bind using search DN \"{}\"", searchBindDN); + return null; + } + try { // Retrieve all DNs associated with the given username