Skip to content
Permalink
Browse files
Merge pull request #31 from fbelzunc/JENKINS-34444
[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 eecff841eb6761c6ba58a0aa183391549be26929
Showing 1 changed file with 1 addition and 1 deletion.
@@ -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());
}

0 comments on commit eecff84

Please sign in to comment.