Skip to content

Commit

Permalink
Merge pull request #31 from fbelzunc/JENKINS-34444
Browse files Browse the repository at this point in the history
[FIXED JENKINS-34444] Test button is reporting managerDN binding is successful but was not able to find any user on the tree
  • Loading branch information
fbelzunc committed Apr 26, 2016
2 parents 73f0ce1 + 1b19366 commit eecff84
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -439,7 +439,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
Attributes userAttributes = new LDAPSearchBuilder(context, toDC(name)).searchOne("(objectClass=user)");
Attributes userAttributes = new LDAPSearchBuilder(context, toDC(name)).subTreeScope().searchOne("(objectClass=user)");
if (userAttributes == null) {
return FormValidation.error(Messages.ActiveDirectorySecurityRealm_NoUsers());
}
Expand Down

0 comments on commit eecff84

Please sign in to comment.