Skip to content

Commit

Permalink
Make test-button work for multidomain configurations
Browse files Browse the repository at this point in the history
When validating the configuration, the domain is split at ',' and
each part will be validated individually. However, the actually bind
will use the original domain string with ',' which doesn't work.
  • Loading branch information
gonzzor committed Nov 23, 2013
1 parent 45dfbcc commit f15bd8e
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -308,7 +308,7 @@ public FormValidation doValidate(@QueryParameter(fixEmpty = true) String domain,
DirContext context = bind(bindName, Secret.toString(password), servers);
try {
// actually do a search to make sure the credential is valid
new LDAPSearchBuilder(context, toDC(domain)).searchOne("(objectClass=user)");
new LDAPSearchBuilder(context, toDC(name)).searchOne("(objectClass=user)");
} finally {
context.close();
}
Expand Down

0 comments on commit f15bd8e

Please sign in to comment.