Skip to content
Snippets Groups Projects
Commit 6b71591b authored by srosse's avatar srosse
Browse files

no-jira: only sync LDAP groups with a member attribute

parent 08b91502
No related branches found
No related tags found
No related merge requests found
...@@ -54,24 +54,26 @@ public class LDAPGroupVisitor implements LDAPVisitor { ...@@ -54,24 +54,26 @@ public class LDAPGroupVisitor implements LDAPVisitor {
Attribute memberAttr = resAttributes.get("member"); Attribute memberAttr = resAttributes.get("member");
Attribute cnAttr = resAttributes.get("cn"); Attribute cnAttr = resAttributes.get("cn");
LDAPGroup group = new LDAPGroup(); if(memberAttr != null) {
Object cn = cnAttr.get(); LDAPGroup group = new LDAPGroup();
if(cn instanceof String) { Object cn = cnAttr.get();
group.setCommonName((String)cn); if(cn instanceof String) {
} group.setCommonName((String)cn);
}
List<String> members = new ArrayList<String>();
try { List<String> members = new ArrayList<String>();
for(NamingEnumeration<?> memberEn = memberAttr.getAll(); memberEn.hasMoreElements(); ) { try {
Object member = memberEn.next(); for(NamingEnumeration<?> memberEn = memberAttr.getAll(); memberEn.hasMoreElements(); ) {
if(member instanceof String) { Object member = memberEn.next();
members.add((String)member); if(member instanceof String) {
members.add((String)member);
}
} }
} catch (NamingException e) {
log.error("", e);
} }
} catch (NamingException e) { group.setMembers(members);
log.error("", e); groups.add(group);
} }
group.setMembers(members);
groups.add(group);
} }
} }
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment