diff --git a/kafka-ui-api/src/main/java/com/provectus/kafka/ui/config/auth/OAuthSecurityConfig.java b/kafka-ui-api/src/main/java/com/provectus/kafka/ui/config/auth/OAuthSecurityConfig.java index d170a7338c..ce0ef4e5e1 100644 --- a/kafka-ui-api/src/main/java/com/provectus/kafka/ui/config/auth/OAuthSecurityConfig.java +++ b/kafka-ui-api/src/main/java/com/provectus/kafka/ui/config/auth/OAuthSecurityConfig.java @@ -104,6 +104,9 @@ public class OAuthSecurityConfig extends AbstractAuthSecurityConfig { final OAuth2ClientProperties props = OAuthPropertiesConverter.convertProperties(properties); final List registrations = new ArrayList<>(OAuth2ClientPropertiesRegistrationAdapter.getClientRegistrations(props).values()); + if (registrations.isEmpty()) { + throw new IllegalArgumentException("OAuth2 authentication is enabled but no providers specified."); + } return new InMemoryReactiveClientRegistrationRepository(registrations); } diff --git a/kafka-ui-api/src/main/java/com/provectus/kafka/ui/config/auth/condition/CognitoCondition.java b/kafka-ui-api/src/main/java/com/provectus/kafka/ui/config/auth/condition/CognitoCondition.java index dc56195ecc..c369985872 100644 --- a/kafka-ui-api/src/main/java/com/provectus/kafka/ui/config/auth/condition/CognitoCondition.java +++ b/kafka-ui-api/src/main/java/com/provectus/kafka/ui/config/auth/condition/CognitoCondition.java @@ -1,13 +1,14 @@ package com.provectus.kafka.ui.config.auth.condition; import com.provectus.kafka.ui.service.rbac.AbstractProviderCondition; +import org.jetbrains.annotations.NotNull; import org.springframework.context.annotation.Condition; import org.springframework.context.annotation.ConditionContext; import org.springframework.core.type.AnnotatedTypeMetadata; public class CognitoCondition extends AbstractProviderCondition implements Condition { @Override - public boolean matches(final ConditionContext context, final AnnotatedTypeMetadata metadata) { + public boolean matches(final ConditionContext context, final @NotNull AnnotatedTypeMetadata metadata) { return getRegisteredProvidersTypes(context.getEnvironment()).stream().anyMatch(a -> a.equalsIgnoreCase("cognito")); } -} \ No newline at end of file +}