Federico Simoncelli has uploaded a new change for review.

Change subject: core: use the ou=People dn for openldap users
......................................................................

core: use the ou=People dn for openldap users

In openldap deployments the ou=People dn is more widely used than
cn=Users,cn=Accounts (more freeipa-like).
Until we wait for this parameter to be configurable let's switch
it to ou=People.

Change-Id: I84707ad92bdd2427f9acb7974149859ed79a751a
Signed-off-by: Federico Simoncelli <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/SimpleLdapTemplateWrapper.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/40/17240/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/SimpleLdapTemplateWrapper.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/SimpleLdapTemplateWrapper.java
index 6e816f9..84475ff 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/SimpleLdapTemplateWrapper.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/SimpleLdapTemplateWrapper.java
@@ -43,7 +43,7 @@
 
         if ( ldapProviderType.equals(LdapProviderType.activeDirectory) ) {
             userDNSb.append("CN=").append(userName).append(",CN=Users");
-        } else if (ldapProviderType.equals(LdapProviderType.rhds)) {
+        } else if (ldapProviderType.equals(LdapProviderType.rhds) || 
ldapProviderType.equals(LdapProviderType.openLdap)) {
             userDNSb.append("uid=").append(userName).append(",ou=People");
         } else {
             
userDNSb.append("uid=").append(userName).append(",cn=Users").append(",cn=Accounts");


-- 
To view, visit http://gerrit.ovirt.org/17240
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I84707ad92bdd2427f9acb7974149859ed79a751a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to