Replace field injection with constructor injection in config classes

Closes spring-projectsgh-7563

Author: wilkinsona

spring-boot-actuator/src/main/java/org/springframework/boot/actuate/autoconfigure/EndpointWebMvcManagementContextConfiguration.java

@@ -16,10 +16,11 @@
 
 package org.springframework.boot.actuate.autoconfigure;
 
+import java.util.Collections;
 import java.util.List;
 import java.util.Set;
 
-import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.beans.factory.ObjectProvider;
 import org.springframework.boot.actuate.condition.ConditionalOnEnabledEndpoint;
 import org.springframework.boot.actuate.endpoint.Endpoint;
 import org.springframework.boot.actuate.endpoint.EnvironmentEndpoint;
@@ -67,17 +68,28 @@
 		EndpointCorsProperties.class })
 public class EndpointWebMvcManagementContextConfiguration {
 
-	@Autowired
-	private HealthMvcEndpointProperties healthMvcEndpointProperties;
+	private final HealthMvcEndpointProperties healthMvcEndpointProperties;
 
-	@Autowired
-	private ManagementServerProperties managementServerProperties;
+	private final ManagementServerProperties managementServerProperties;
 
-	@Autowired
-	private EndpointCorsProperties corsProperties;
+	private final EndpointCorsProperties corsProperties;
 
-	@Autowired(required = false)
-	private List<EndpointHandlerMappingCustomizer> mappingCustomizers;
+	private final List<EndpointHandlerMappingCustomizer> mappingCustomizers;
+
+	public EndpointWebMvcManagementContextConfiguration(
+			HealthMvcEndpointProperties healthMvcEndpointProperties,
+			ManagementServerProperties managementServerProperties,
+			EndpointCorsProperties corsProperties,
+			ObjectProvider<List<EndpointHandlerMappingCustomizer>> mappingCustomizersProvider) {
+		this.healthMvcEndpointProperties = healthMvcEndpointProperties;
+		this.managementServerProperties = managementServerProperties;
+		this.corsProperties = corsProperties;
+		List<EndpointHandlerMappingCustomizer> providedCustomizers = mappingCustomizersProvider
+				.getIfAvailable();
+		this.mappingCustomizers = providedCustomizers == null
+				? Collections.<EndpointHandlerMappingCustomizer>emptyList()
+				: providedCustomizers;
+	}
 
 	@Bean
 	@ConditionalOnMissingBean
@@ -87,10 +99,8 @@ public EndpointHandlerMapping endpointHandlerMapping() {
 		EndpointHandlerMapping mapping = new EndpointHandlerMapping(endpoints,
 				corsConfiguration);
 		mapping.setPrefix(this.managementServerProperties.getContextPath());
-		if (this.mappingCustomizers != null) {
-			for (EndpointHandlerMappingCustomizer customizer : this.mappingCustomizers) {
-				customizer.customize(mapping);
-			}
+		for (EndpointHandlerMappingCustomizer customizer : this.mappingCustomizers) {
+			customizer.customize(mapping);
 		}
 		return mapping;
 	}

spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/integration/IntegrationAutoConfiguration.java

@@ -21,7 +21,7 @@
 import org.springframework.beans.BeansException;
 import org.springframework.beans.factory.BeanFactory;
 import org.springframework.beans.factory.BeanFactoryAware;
-import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.beans.factory.ObjectProvider;
 import org.springframework.beans.factory.annotation.Qualifier;
 import org.springframework.boot.autoconfigure.AutoConfigureAfter;
 import org.springframework.boot.autoconfigure.condition.ConditionalOnClass;
@@ -66,13 +66,16 @@ protected static class IntegrationConfiguration {
 	protected static class IntegrationJmxConfiguration
 			implements EnvironmentAware, BeanFactoryAware {
 
+		private final IntegrationManagementConfigurer configurer;
+
 		private BeanFactory beanFactory;
 
 		private RelaxedPropertyResolver propertyResolver;
 
-		@Autowired(required = false)
-		@Qualifier(IntegrationManagementConfigurer.MANAGEMENT_CONFIGURER_NAME)
-		private IntegrationManagementConfigurer configurer;
+		protected IntegrationJmxConfiguration(
+				@Qualifier(IntegrationManagementConfigurer.MANAGEMENT_CONFIGURER_NAME) ObjectProvider<IntegrationManagementConfigurer> configurerProvider) {
+			this.configurer = configurerProvider.getIfAvailable();
+		}
 
 		@Override
 		public void setBeanFactory(BeanFactory beanFactory) throws BeansException {