necouchman commented on a change in pull request #496: GUACAMOLE-996
URL: https://github.com/apache/guacamole-client/pull/496#discussion_r403391139
##########
File path:
extensions/guacamole-auth-ldap/src/main/java/org/apache/guacamole/auth/ldap/group/UserGroupService.java
##########
@@ -79,17 +79,17 @@
* @throws GuacamoleException
* If guacamole.properties cannot be parsed.
*/
- private ExprNode getGroupSearchFilter() throws GuacamoleException {
+ private ExprNode buildGroupSearchFilter() throws GuacamoleException {
// Explicitly exclude guacConfigGroup object class only if it should
// be assumed to be defined (query may fail due to no such object
// class existing otherwise)
if (confService.getConfigurationBaseDN() != null)
return new NotNode(new
EqualityNode("objectClass","guacConfigGroup"));
- // Read any object as a group if LDAP is not being used for connection
+ // Read objects from LDAP with filter defined by
"ldap-group-search-filter" as a group if LDAP is not being used for connection
Review comment:
This should probably just be made a multi-line comment - it's pretty long.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
