From c530865a9560c6639b3541ebe7d8f6af8e53453b Mon Sep 17 00:00:00 2001 From: Michael Jumper Date: Mon, 2 Nov 2015 17:50:49 -0800 Subject: [PATCH] GUAC-1342: Clarify username attribute warning. --- .../glyptodon/guacamole/auth/ldap/user/UserService.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/extensions/guacamole-auth-ldap/src/main/java/org/glyptodon/guacamole/auth/ldap/user/UserService.java b/extensions/guacamole-auth-ldap/src/main/java/org/glyptodon/guacamole/auth/ldap/user/UserService.java index 5761b6184..1f47d4c00 100644 --- a/extensions/guacamole-auth-ldap/src/main/java/org/glyptodon/guacamole/auth/ldap/user/UserService.java +++ b/extensions/guacamole-auth-ldap/src/main/java/org/glyptodon/guacamole/auth/ldap/user/UserService.java @@ -36,6 +36,7 @@ import org.glyptodon.guacamole.auth.ldap.ConfigurationService; import org.glyptodon.guacamole.auth.ldap.EscapingService; import org.glyptodon.guacamole.GuacamoleException; import org.glyptodon.guacamole.GuacamoleServerException; +import org.glyptodon.guacamole.auth.ldap.LDAPGuacamoleProperties; import org.glyptodon.guacamole.net.auth.User; import org.glyptodon.guacamole.net.auth.simple.SimpleUser; import org.slf4j.Logger; @@ -296,7 +297,11 @@ public class UserService { // We need exactly one base DN to derive the user DN if (usernameAttributes.size() != 1) { - logger.warn("Cannot directly derive user DN when multiple username attributes are specified"); + logger.warn(String.format("Cannot directly derive user DN when " + + "multiple username attributes are specified. Please " + + "define an LDAP search DN using the \"%s\" property " + + "in your \"guacamole.properties\".", + LDAPGuacamoleProperties.LDAP_SEARCH_BIND_DN.getName())); return null; }