DaanHoogland commented on code in PR #11436:
URL: https://github.com/apache/cloudstack/pull/11436#discussion_r2607121662
##########
plugins/user-authenticators/ldap/src/main/java/org/apache/cloudstack/api/command/LdapListUsersCmd.java:
##########
@@ -433,10 +414,10 @@ public List<LdapUserResponse>
filterPotentialImport(List<LdapUserResponse> input
logger.trace("should be filtering potential imports!!!");
}
// functional possibility do not add only users not yet in cloudstack
but include users that would be moved if they are so in ldap?
- // this means if they are part of a account linked to an ldap group/ou
+ // this means if they are part of an account linked to a ldap group/ou
Review Comment:
```suggestion
// This means if they are part of an Account linked to an LDAP
Group/OU
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
