Skip to content

Commit 31b0dec

Browse files
committed
Throw a warning when no value is set
1 parent 4025436 commit 31b0dec

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

api/src/main/java/io/kafbat/ui/config/auth/LdapSecurityConfig.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
import java.util.stream.Stream;
1313
import lombok.RequiredArgsConstructor;
1414
import lombok.extern.slf4j.Slf4j;
15+
import org.apache.commons.lang3.StringUtils;
1516
import org.springframework.beans.factory.annotation.Autowired;
1617
import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty;
1718
import org.springframework.boot.context.properties.EnableConfigurationProperties;
@@ -163,6 +164,10 @@ public SecurityWebFilterChain configureLdap(ServerHttpSecurity http) {
163164
}
164165

165166
private ActiveDirectoryLdapAuthenticationProvider activeDirectoryProvider(LdapAuthoritiesPopulator populator) {
167+
if (StringUtils.isBlank(props.getActiveDirectoryDomain())) {
168+
throw new IllegalArgumentException("Active Directory domain is required but not specified");
169+
}
170+
166171
ActiveDirectoryLdapAuthenticationProvider provider = new ActiveDirectoryLdapAuthenticationProvider(
167172
props.getActiveDirectoryDomain(),
168173
props.getUrls()

0 commit comments

Comments
 (0)