Skip to content

Commit 95d5e35

Browse files
wilkinsonawilkinsona
committed
Provide @ConditionalOn… annotations for all concrete conditions
Closes gh-41044
1 parent 437c259 commit 95d5e35

File tree

20 files changed

+244
-43
lines changed

20 files changed

+244
-43
lines changed

‎buildSrc/src/main/java/org/springframework/boot/build/architecture/ArchitectureRules.java‎

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@
3434
import com.tngtech.archunit.core.domain.JavaClass;
3535
import com.tngtech.archunit.core.domain.JavaClass.Predicates;
3636
import com.tngtech.archunit.core.domain.JavaMethod;
37+
import com.tngtech.archunit.core.domain.JavaModifier;
3738
import com.tngtech.archunit.core.domain.JavaParameter;
3839
import com.tngtech.archunit.core.domain.JavaType;
3940
import com.tngtech.archunit.core.domain.properties.CanBeAnnotated;
@@ -94,6 +95,7 @@ static List<ArchRule> standard() {
9495
rules.add(enumSourceShouldNotSpecifyOnlyATypeThatIsTheSameAsMethodParameterType());
9596
rules.add(classLevelConfigurationPropertiesShouldNotSpecifyOnlyPrefixAttribute());
9697
rules.add(methodLevelConfigurationPropertiesShouldNotSpecifyOnlyPrefixAttribute());
98+
rules.add(conditionsShouldNotBePublic());
9799
return List.copyOf(rules);
98100
}
99101

@@ -290,6 +292,20 @@ private static void notSpecifyOnlyPrefixAttributeOfConfigurationProperties(HasAn
290292
}
291293
}
292294

295+
private static ArchRule conditionsShouldNotBePublic() {
296+
String springBootCondition = "org.springframework.boot.autoconfigure.condition.SpringBootCondition";
297+
return ArchRuleDefinition.noClasses()
298+
.that()
299+
.areAssignableTo(springBootCondition)
300+
.and()
301+
.doNotHaveModifier(JavaModifier.ABSTRACT)
302+
.and()
303+
.areNotAnnotatedWith(Deprecated.class)
304+
.should()
305+
.bePublic()
306+
.allowEmptyShould(true);
307+
}
308+
293309
private static boolean containsOnlySingleType(JavaType[] types, JavaType type) {
294310
return types.length == 1 && type.equals(types[0]);
295311
}

‎spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/condition/OnPropertyListCondition.java‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2012-2019 the original author or authors.
2+
* Copyright 2012-2025 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -34,7 +34,7 @@
3434
* @author Stephane Nicoll
3535
* @since 2.0.5
3636
*/
37-
public class OnPropertyListCondition extends SpringBootCondition {
37+
public abstract class OnPropertyListCondition extends SpringBootCondition {
3838

3939
private static final Bindable<List<String>> STRING_LIST = Bindable.listOf(String.class);
4040

‎spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/security/oauth2/client/ClientsConfiguredCondition.java‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2012-2024 the original author or authors.
2+
* Copyright 2012-2025 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -35,7 +35,10 @@
3535
*
3636
* @author Madhura Bhave
3737
* @since 2.1.0
38+
* @deprecated since 3.5.0 for removal in 3.7.0 in favor of
39+
* {@link ConditionalOnOAuth2ClientRegistrationProperties @ConditionalOnOAuth2ClientRegistrationConfigured}
3840
*/
41+
@Deprecated(since = "3.5.0", forRemoval = true)
3942
public class ClientsConfiguredCondition extends SpringBootCondition {
4043

4144
private static final Bindable<Map<String, OAuth2ClientProperties.Registration>> STRING_REGISTRATION_MAP = Bindable

‎spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/security/oauth2/client/ConditionalOnOAuth2ClientRegistrationProperties.java‎

Lines changed: 41 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,41 @@
1+
/*
2+
* Copyright 2012-2025 the original author or authors.
3+
*
4+
* Licensed under the Apache License, Version 2.0 (the "License");
5+
* you may not use this file except in compliance with the License.
6+
* You may obtain a copy of the License at
7+
*
8+
* https://www.apache.org/licenses/LICENSE-2.0
9+
*
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS,
12+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
* See the License for the specific language governing permissions and
14+
* limitations under the License.
15+
*/
16+
17+
package org.springframework.boot.autoconfigure.security.oauth2.client;
18+
19+
import java.lang.annotation.Documented;
20+
import java.lang.annotation.ElementType;
21+
import java.lang.annotation.Retention;
22+
import java.lang.annotation.RetentionPolicy;
23+
import java.lang.annotation.Target;
24+
25+
import org.springframework.context.annotation.Conditional;
26+
27+
/**
28+
* Condition that matches if any {@code spring.security.oauth2.client.registration}
29+
* properties are defined.
30+
*
31+
* @author Andy Wilkinson
32+
* @since 3.5.0
33+
*/
34+
@SuppressWarnings("removal")
35+
@Retention(RetentionPolicy.RUNTIME)
36+
@Target({ ElementType.TYPE, ElementType.METHOD })
37+
@Documented
38+
@Conditional(ClientsConfiguredCondition.class)
39+
public @interface ConditionalOnOAuth2ClientRegistrationProperties {
40+
41+
}

‎spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/security/oauth2/client/reactive/ReactiveOAuth2ClientConfigurations.java‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2012-2023 the original author or authors.
2+
* Copyright 2012-2025 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -22,11 +22,10 @@
2222
import org.springframework.boot.autoconfigure.condition.ConditionalOnBean;
2323
import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
2424
import org.springframework.boot.autoconfigure.condition.ConditionalOnWebApplication;
25-
import org.springframework.boot.autoconfigure.security.oauth2.client.ClientsConfiguredCondition;
25+
import org.springframework.boot.autoconfigure.security.oauth2.client.ConditionalOnOAuth2ClientRegistrationProperties;
2626
import org.springframework.boot.autoconfigure.security.oauth2.client.OAuth2ClientProperties;
2727
import org.springframework.boot.autoconfigure.security.oauth2.client.OAuth2ClientPropertiesMapper;
2828
import org.springframework.context.annotation.Bean;
29-
import org.springframework.context.annotation.Conditional;
3029
import org.springframework.context.annotation.Configuration;
3130
import org.springframework.security.config.web.server.ServerHttpSecurity;
3231
import org.springframework.security.oauth2.client.InMemoryReactiveOAuth2AuthorizedClientService;
@@ -48,7 +47,7 @@
4847
class ReactiveOAuth2ClientConfigurations {
4948

5049
@Configuration(proxyBeanMethods = false)
51-
@Conditional(ClientsConfiguredCondition.class)
50+
@ConditionalOnOAuth2ClientRegistrationProperties
5251
@ConditionalOnMissingBean(ReactiveClientRegistrationRepository.class)
5352
static class ReactiveClientRegistrationRepositoryConfiguration {
5453

‎spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/security/oauth2/client/servlet/OAuth2ClientRegistrationRepositoryConfiguration.java‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2012-2023 the original author or authors.
2+
* Copyright 2012-2025 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -20,12 +20,11 @@
2020
import java.util.List;
2121

2222
import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
23-
import org.springframework.boot.autoconfigure.security.oauth2.client.ClientsConfiguredCondition;
23+
import org.springframework.boot.autoconfigure.security.oauth2.client.ConditionalOnOAuth2ClientRegistrationProperties;
2424
import org.springframework.boot.autoconfigure.security.oauth2.client.OAuth2ClientProperties;
2525
import org.springframework.boot.autoconfigure.security.oauth2.client.OAuth2ClientPropertiesMapper;
2626
import org.springframework.boot.context.properties.EnableConfigurationProperties;
2727
import org.springframework.context.annotation.Bean;
28-
import org.springframework.context.annotation.Conditional;
2928
import org.springframework.context.annotation.Configuration;
3029
import org.springframework.security.oauth2.client.registration.ClientRegistration;
3130
import org.springframework.security.oauth2.client.registration.ClientRegistrationRepository;
@@ -39,7 +38,7 @@
3938
*/
4039
@Configuration(proxyBeanMethods = false)
4140
@EnableConfigurationProperties(OAuth2ClientProperties.class)
42-
@Conditional(ClientsConfiguredCondition.class)
41+
@ConditionalOnOAuth2ClientRegistrationProperties
4342
class OAuth2ClientRegistrationRepositoryConfiguration {
4443

4544
@Bean

‎spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/security/oauth2/resource/ConditionalOnIssuerLocationJwtDecoder.java‎

Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
/*
2+
* Copyright 2012-2025 the original author or authors.
3+
*
4+
* Licensed under the Apache License, Version 2.0 (the "License");
5+
* you may not use this file except in compliance with the License.
6+
* You may obtain a copy of the License at
7+
*
8+
* https://www.apache.org/licenses/LICENSE-2.0
9+
*
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS,
12+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
* See the License for the specific language governing permissions and
14+
* limitations under the License.
15+
*/
16+
17+
package org.springframework.boot.autoconfigure.security.oauth2.resource;
18+
19+
import java.lang.annotation.Documented;
20+
import java.lang.annotation.ElementType;
21+
import java.lang.annotation.Retention;
22+
import java.lang.annotation.RetentionPolicy;
23+
import java.lang.annotation.Target;
24+
25+
import org.springframework.context.annotation.Conditional;
26+
import org.springframework.security.oauth2.jwt.NimbusJwtDecoder;
27+
28+
/**
29+
* Condition that matches when an {@link NimbusJwtDecoder#withIssuerLocation
30+
* issuer-location-based JWT decoder} should be used.
31+
*
32+
* @author Andy Wilkinson
33+
* @since 3.5.0
34+
*/
35+
@SuppressWarnings("removal")
36+
@Retention(RetentionPolicy.RUNTIME)
37+
@Target({ ElementType.TYPE, ElementType.METHOD })
38+
@Documented
39+
@Conditional(IssuerUriCondition.class)
40+
public @interface ConditionalOnIssuerLocationJwtDecoder {
41+
42+
}

‎spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/security/oauth2/resource/ConditionalOnPublicKeyJwtDecoder.java‎

Lines changed: 42 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,42 @@
1+
/*
2+
* Copyright 2012-2025 the original author or authors.
3+
*
4+
* Licensed under the Apache License, Version 2.0 (the "License");
5+
* you may not use this file except in compliance with the License.
6+
* You may obtain a copy of the License at
7+
*
8+
* https://www.apache.org/licenses/LICENSE-2.0
9+
*
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS,
12+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
* See the License for the specific language governing permissions and
14+
* limitations under the License.
15+
*/
16+
17+
package org.springframework.boot.autoconfigure.security.oauth2.resource;
18+
19+
import java.lang.annotation.Documented;
20+
import java.lang.annotation.ElementType;
21+
import java.lang.annotation.Retention;
22+
import java.lang.annotation.RetentionPolicy;
23+
import java.lang.annotation.Target;
24+
25+
import org.springframework.context.annotation.Conditional;
26+
import org.springframework.security.oauth2.jwt.NimbusJwtDecoder;
27+
28+
/**
29+
* Condition that matches when a {@link NimbusJwtDecoder#withPublicKey public-key-based
30+
* JWT decoder} should be used.
31+
*
32+
* @author Andy Wilkinson
33+
* @since 3.5.0
34+
*/
35+
@SuppressWarnings("removal")
36+
@Retention(RetentionPolicy.RUNTIME)
37+
@Target({ ElementType.TYPE, ElementType.METHOD })
38+
@Documented
39+
@Conditional(KeyValueCondition.class)
40+
public @interface ConditionalOnPublicKeyJwtDecoder {
41+
42+
}

‎spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/security/oauth2/resource/IssuerUriCondition.java‎

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2012-2020 the original author or authors.
2+
* Copyright 2012-2025 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -30,18 +30,21 @@
3030
*
3131
* @author Artsiom Yudovin
3232
* @since 2.1.0
33+
* @deprecated since 3.5.0 for removal in 3.7.0 in favor of
34+
* {@link ConditionalOnIssuerLocationJwtDecoder @ConditionalOnIssuerLocationJwtDecoder}
3335
*/
36+
@Deprecated(since = "3.5.0", forRemoval = true)
3437
public class IssuerUriCondition extends SpringBootCondition {
3538

3639
@Override
3740
public ConditionOutcome getMatchOutcome(ConditionContext context, AnnotatedTypeMetadata metadata) {
3841
ConditionMessage.Builder message = ConditionMessage.forCondition("OpenID Connect Issuer URI Condition");
3942
Environment environment = context.getEnvironment();
4043
String issuerUri = environment.getProperty("spring.security.oauth2.resourceserver.jwt.issuer-uri");
41-
String jwkSetUri = environment.getProperty("spring.security.oauth2.resourceserver.jwt.jwk-set-uri");
4244
if (!StringUtils.hasText(issuerUri)) {
4345
return ConditionOutcome.noMatch(message.didNotFind("issuer-uri property").atAll());
4446
}
47+
String jwkSetUri = environment.getProperty("spring.security.oauth2.resourceserver.jwt.jwk-set-uri");
4548
if (StringUtils.hasText(jwkSetUri)) {
4649
return ConditionOutcome.noMatch(message.found("jwk-set-uri property").items(jwkSetUri));
4750
}

‎spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/security/oauth2/resource/KeyValueCondition.java‎

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2012-2023 the original author or authors.
2+
* Copyright 2012-2025 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -29,7 +29,10 @@
2929
*
3030
* @author Madhura Bhave
3131
* @since 2.2.0
32+
* @deprecated since 3.5.0 for removal in 3.7.0 in favor of
33+
* {@link ConditionalOnPublicKeyJwtDecoder @ConditionalOnPublicKeyJwtDecoder}
3234
*/
35+
@Deprecated(since = "3.5.0", forRemoval = true)
3336
public class KeyValueCondition extends SpringBootCondition {
3437

3538
@Override
@@ -41,11 +44,11 @@ public ConditionOutcome getMatchOutcome(ConditionContext context, AnnotatedTypeM
4144
if (!StringUtils.hasText(publicKeyLocation)) {
4245
return ConditionOutcome.noMatch(message.didNotFind("public-key-location property").atAll());
4346
}
44-
String issuerUri = environment.getProperty("spring.security.oauth2.resourceserver.jwt.issuer-uri");
4547
String jwkSetUri = environment.getProperty("spring.security.oauth2.resourceserver.jwt.jwk-set-uri");
4648
if (StringUtils.hasText(jwkSetUri)) {
4749
return ConditionOutcome.noMatch(message.found("jwk-set-uri property").items(jwkSetUri));
4850
}
51+
String issuerUri = environment.getProperty("spring.security.oauth2.resourceserver.jwt.issuer-uri");
4952
if (StringUtils.hasText(issuerUri)) {
5053
return ConditionOutcome.noMatch(message.found("issuer-uri property").items(issuerUri));
5154
}

0 commit comments

Comments
 (0)