From: Mathieu Baudier <mbaudier@argeo.org>
Date: Mon, 9 Jan 2017 10:25:50 +0000 (+0100)
Subject: Fix issue with lower case object classes (typically in IPA)
X-Git-Tag: argeo-commons-2.1.56~13
X-Git-Url: http://git.argeo.org/?a=commitdiff_plain;h=c6774c9dacea9aeeeaa594abf56a3331289bd3ba;p=lgpl%2Fargeo-commons.git

Fix issue with lower case object classes (typically in IPA)
---

diff --git a/org.argeo.enterprise/src/org/argeo/osgi/useradmin/LdapUserAdmin.java b/org.argeo.enterprise/src/org/argeo/osgi/useradmin/LdapUserAdmin.java
index 456342e04..978b9a669 100644
--- a/org.argeo.enterprise/src/org/argeo/osgi/useradmin/LdapUserAdmin.java
+++ b/org.argeo.enterprise/src/org/argeo/osgi/useradmin/LdapUserAdmin.java
@@ -120,7 +120,7 @@ public class LdapUserAdmin extends AbstractUserDirectory {
 				throw new UserDirectoryException("Unsupported LDAP type for " + name);
 			return res;
 		} catch (NamingException e) {
-			log.error("Cannot get role: "+e.getMessage());
+			log.error("Cannot get role: " + e.getMessage());
 			return null;
 		}
 	}
@@ -144,9 +144,11 @@ public class LdapUserAdmin extends AbstractUserDirectory {
 				Attribute objectClassAttr = attrs.get(objectClass.name());
 				LdapName dn = toDn(searchBase, searchResult);
 				LdifUser role;
-				if (objectClassAttr.contains(getGroupObjectClass()))
+				if (objectClassAttr.contains(getGroupObjectClass())
+						|| objectClassAttr.contains(getGroupObjectClass().toLowerCase()))
 					role = new LdifGroup(this, dn, attrs);
-				else if (objectClassAttr.contains(getUserObjectClass()))
+				else if (objectClassAttr.contains(getUserObjectClass())
+						|| objectClassAttr.contains(getUserObjectClass().toLowerCase()))
 					role = new LdifUser(this, dn, attrs);
 				else {
 					log.warn("Unsupported LDAP type for " + searchResult.getName());