Github user necouchman commented on a diff in the pull request:
https://github.com/apache/guacamole-client/pull/308#discussion_r206498210
--- Diff:
extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/connection/ConnectionService.java
---
@@ -245,7 +245,10 @@ private String getConnectionSearchFilter(String userDN,
StringBuilder connectionSearchFilter = new StringBuilder();
// Add the prefix to the search filter, prefix filter searches for
guacConfigGroups with the userDN as the member attribute value
-
connectionSearchFilter.append("(&(objectClass=guacConfigGroup)(|(member=");
+ connectionSearchFilter.append("(&(objectClass=guacConfigGroup)");
+ connectionSearchFilter.append("(|(");
+ connectionSearchFilter.append(confService.getMemberAttribute());
--- End diff --
Escaped.
---
