Skip to content

Commit a1ab420

Browse files
author
Tom Nieuwenhuijs
committed
Add a default port to all configured domain controllers except to only the last one
1 parent 7ad2ff4 commit a1ab420

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

src/main/java/hudson/plugins/active_directory/ActiveDirectorySecurityRealm.java

+8-1
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@
5050
import org.acegisecurity.userdetails.UserDetailsService;
5151
import org.acegisecurity.userdetails.UsernameNotFoundException;
5252
import org.apache.commons.io.IOUtils;
53+
import org.apache.commons.lang.StringUtils;
5354
import org.kohsuke.stapler.DataBoundConstructor;
5455
import org.kohsuke.stapler.QueryParameter;
5556
import org.kohsuke.stapler.StaplerRequest;
@@ -171,7 +172,13 @@ public ActiveDirectorySecurityRealm(String domain, String site, String bindName,
171172
// append default port if not specified
172173
server = fixEmpty(server);
173174
if (server != null) {
174-
if (!server.contains(":")) server += ":3268";
175+
String[] servers = server.split(",");
176+
for (int i = 0; i < servers.length; i++) {
177+
if (!servers[i].contains(":")) {
178+
servers[i] += ":3268";
179+
}
180+
}
181+
server = StringUtils.join(servers, ",");
175182
}
176183

177184
this.server = server;

0 commit comments

Comments
 (0)