diff --git a/extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/ConfigurationService.java b/extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/ConfigurationService.java index f29d8f106..e546414d6 100644 --- a/extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/ConfigurationService.java +++ b/extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/ConfigurationService.java @@ -241,17 +241,23 @@ public class ConfigurationService { "never" ); - if (derefAliases == "always") + if (derefAliases.equals("always")) return 3; - else if (derefAliases == "finding") + else if (derefAliases.equals("finding")) return 2; - else if (derefAliases == "searching") + else if (derefAliases.equals("searching")) return 1; - else + else if (derefAliases.equals("never")) return 0; + + else { + logger.error("Invalid value given for ldap-dereference-aliases."); + logger.debug("Received {} but expected one of the following: always, finding, searching, never.", derefAliases); + throw new GuacamoleException("Invalid valid for ldap-dereference-aliases."); + } }