Skip to content
Permalink
Browse files
Fix for MRM-1873 which broke AD group assignments
Escaping only needed in the LDAP filter string
  • Loading branch information
svogt committed May 28, 2015
1 parent fc28ad1 commit d0bc52fff1d8901decff5b16dd2e6ed07bb67354
Showing 1 changed file with 1 addition and 3 deletions.
@@ -392,11 +392,9 @@ public List<String> getGroups( String username, DirContext context )
groupEntry = builder.toString();
}

groupEntry = Rdn.escapeValue(groupEntry);

String filter =
new StringBuilder().append( "(&" ).append( "(objectClass=" + getLdapGroupClass() + ")" ).append(
"(" ).append( getLdapGroupMember() ).append( "=" ).append( groupEntry ).append( ")" ).append(
"(" ).append( getLdapGroupMember() ).append( "=" ).append( Rdn.escapeValue(groupEntry) ).append( ")" ).append(
")" ).toString();

log.debug( "filter: {}", filter );

0 comments on commit d0bc52f

Please sign in to comment.