From 80bb4aaa57698ee8d56d77155d88c5260726c748 Mon Sep 17 00:00:00 2001 From: Roman Zabaluev Date: Fri, 12 Jul 2024 19:23:03 +0300 Subject: [PATCH] Clarify url Parameter Usage in AD Provider Closes gh-7760 --- .../ad/ActiveDirectoryLdapAuthenticationProvider.java | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/ldap/src/main/java/org/springframework/security/ldap/authentication/ad/ActiveDirectoryLdapAuthenticationProvider.java b/ldap/src/main/java/org/springframework/security/ldap/authentication/ad/ActiveDirectoryLdapAuthenticationProvider.java index 4dcb42aee08..d69868b0ed9 100644 --- a/ldap/src/main/java/org/springframework/security/ldap/authentication/ad/ActiveDirectoryLdapAuthenticationProvider.java +++ b/ldap/src/main/java/org/springframework/security/ldap/authentication/ad/ActiveDirectoryLdapAuthenticationProvider.java @@ -136,9 +136,11 @@ public final class ActiveDirectoryLdapAuthenticationProvider extends AbstractLda ContextFactory contextFactory = new ContextFactory(); /** - * @param domain the domain name (may be null or empty) - * @param url an LDAP url (or multiple URLs) - * @param rootDn the root DN (may be null or empty) + * @param domain the domain name (can be null or empty) + * @param url an LDAP url (or multiple space-delimited URLs). + * @param rootDn the root DN (can be null or empty) + * @see JNDI + * URL format documentation */ public ActiveDirectoryLdapAuthenticationProvider(String domain, String url, String rootDn) { Assert.isTrue(StringUtils.hasText(url), "Url cannot be empty");