[
https://issues.apache.org/jira/browse/CLOUDSTACK-8596?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14636247#comment-14636247
]
ASF GitHub Bot commented on CLOUDSTACK-8596:
--------------------------------------------
Github user DaanHoogland commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/609#discussion_r35181833
--- Diff:
plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapManagerImpl.java
---
@@ -57,17 +57,22 @@
private LdapContextFactory _ldapContextFactory;
@Inject
- private LdapUserManager _ldapUserManager;
+ private LdapConfiguration _ldapConfiguration;
+
+ @Inject LdapUserManagerFactory _ldapUserManagerFactory;
+
public LdapManagerImpl() {
super();
}
- public LdapManagerImpl(final LdapConfigurationDao
ldapConfigurationDao, final LdapContextFactory ldapContextFactory, final
LdapUserManager ldapUserManager) {
+ public LdapManagerImpl(final LdapConfigurationDao
ldapConfigurationDao, final LdapContextFactory ldapContextFactory, final
LdapUserManagerFactory ldapUserManagerFactory,
+ final LdapConfiguration ldapConfiguration) {
--- End diff --
why inject and pass into constructor? is the ldap-provider-selector going
to change on runtime?
> [LDAP] Nested groups, ability of recursively querying nested groups
> -------------------------------------------------------------------
>
> Key: CLOUDSTACK-8596
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-8596
> Project: CloudStack
> Issue Type: Improvement
> Security Level: Public(Anyone can view this level - this is the
> default.)
> Reporter: Rajani Karuturi
> Assignee: Rajani Karuturi
>
> Nested group imports
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
