Starting building against Spring Framework 5.3.12 snapshots

This reverts commit 3969e63, reversing
changes made to f01d086.

Closes gh-28385
See gh-28252

Author: wilkinsona

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/reactive/WebFluxAutoConfiguration.java

@@ -17,8 +17,6 @@
 package org.springframework.boot.autoconfigure.web.reactive;
 
 import java.time.Duration;
-import java.util.ArrayList;
-import java.util.List;
 import java.util.function.Supplier;
 
 import org.apache.commons.logging.Log;
@@ -53,7 +51,6 @@
 import org.springframework.context.annotation.Import;
 import org.springframework.core.Ordered;
 import org.springframework.core.annotation.Order;
-import org.springframework.core.io.ResourceLoader;
 import org.springframework.format.FormatterRegistry;
 import org.springframework.format.support.FormattingConversionService;
 import org.springframework.http.codec.ServerCodecConfigurer;
@@ -156,21 +153,18 @@ public static class WebFluxConfig implements WebFluxConfigurer {
 
 		private final ObjectProvider<ViewResolver> viewResolvers;
 
-		private final ResourceLoader resourceLoader;
-
 		public WebFluxConfig(WebProperties webProperties, WebFluxProperties webFluxProperties,
 				ListableBeanFactory beanFactory, ObjectProvider<HandlerMethodArgumentResolver> resolvers,
 				ObjectProvider<CodecCustomizer> codecCustomizers,
 				ObjectProvider<ResourceHandlerRegistrationCustomizer> resourceHandlerRegistrationCustomizer,
-				ObjectProvider<ViewResolver> viewResolvers, ResourceLoader resourceLoader) {
+				ObjectProvider<ViewResolver> viewResolvers) {
 			this.resourceProperties = webProperties.getResources();
 			this.webFluxProperties = webFluxProperties;
 			this.beanFactory = beanFactory;
 			this.argumentResolvers = resolvers;
 			this.codecCustomizers = codecCustomizers;
 			this.resourceHandlerRegistrationCustomizer = resourceHandlerRegistrationCustomizer.getIfAvailable();
 			this.viewResolvers = viewResolvers;
-			this.resourceLoader = resourceLoader;
 		}
 
 		@Override
@@ -190,28 +184,17 @@ public void addResourceHandlers(ResourceHandlerRegistry registry) {
 				return;
 			}
 			if (!registry.hasMappingForPattern("/webjars/**")) {
-				String webjarsLocation = "classpath:/META-INF/resources/webjars/";
-				if (this.resourceLoader.getResource(webjarsLocation).exists()) {
-					ResourceHandlerRegistration registration = registry.addResourceHandler("/webjars/**")
-							.addResourceLocations(webjarsLocation);
-					configureResourceCaching(registration);
-					customizeResourceHandlerRegistration(registration);
-				}
+				ResourceHandlerRegistration registration = registry.addResourceHandler("/webjars/**")
+						.addResourceLocations("classpath:/META-INF/resources/webjars/");
+				configureResourceCaching(registration);
+				customizeResourceHandlerRegistration(registration);
 			}
 			String staticPathPattern = this.webFluxProperties.getStaticPathPattern();
 			if (!registry.hasMappingForPattern(staticPathPattern)) {
-				List<String> foundLocations = new ArrayList<>();
-				for (String staticLocation : this.resourceProperties.getStaticLocations()) {
-					if (this.resourceLoader.getResource(staticLocation).exists()) {
-						foundLocations.add(staticLocation);
-					}
-				}
-				if (!foundLocations.isEmpty()) {
-					ResourceHandlerRegistration registration = registry.addResourceHandler(staticPathPattern)
-							.addResourceLocations(foundLocations.toArray(new String[0]));
-					configureResourceCaching(registration);
-					customizeResourceHandlerRegistration(registration);
-				}
+				ResourceHandlerRegistration registration = registry.addResourceHandler(staticPathPattern)
+						.addResourceLocations(this.resourceProperties.getStaticLocations());
+				configureResourceCaching(registration);
+				customizeResourceHandlerRegistration(registration);
 			}
 		}
 

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/servlet/WebMvcAutoConfiguration.java

@@ -204,23 +204,20 @@ public static class WebMvcAutoConfigurationAdapter implements WebMvcConfigurer,
 
 		private final ResourceHandlerRegistrationCustomizer resourceHandlerRegistrationCustomizer;
 
-		private final ResourceLoader resourceLoader;
-
 		private ServletContext servletContext;
 
 		public WebMvcAutoConfigurationAdapter(WebProperties webProperties, WebMvcProperties mvcProperties,
 				ListableBeanFactory beanFactory, ObjectProvider<HttpMessageConverters> messageConvertersProvider,
 				ObjectProvider<ResourceHandlerRegistrationCustomizer> resourceHandlerRegistrationCustomizerProvider,
 				ObjectProvider<DispatcherServletPath> dispatcherServletPath,
-				ObjectProvider<ServletRegistrationBean<?>> servletRegistrations, ResourceLoader resourceLoader) {
+				ObjectProvider<ServletRegistrationBean<?>> servletRegistrations) {
 			this.resourceProperties = webProperties.getResources();
 			this.mvcProperties = mvcProperties;
 			this.beanFactory = beanFactory;
 			this.messageConvertersProvider = messageConvertersProvider;
 			this.resourceHandlerRegistrationCustomizer = resourceHandlerRegistrationCustomizerProvider.getIfAvailable();
 			this.dispatcherServletPath = dispatcherServletPath;
 			this.servletRegistrations = servletRegistrations;
-			this.resourceLoader = resourceLoader;
 			this.mvcProperties.checkConfiguration();
 		}
 
@@ -337,11 +334,7 @@ public void addResourceHandlers(ResourceHandlerRegistry registry) {
 				logger.debug("Default resource handling disabled");
 				return;
 			}
-			Resource webjarsLocationResource = this.resourceLoader
-					.getResource("classpath:/META-INF/resources/webjars/");
-			if (webjarsLocationResource.exists()) {
-				addResourceHandler(registry, "/webjars/**", webjarsLocationResource);
-			}
+			addResourceHandler(registry, "/webjars/**", "classpath:/META-INF/resources/webjars/");
 			addResourceHandler(registry, this.mvcProperties.getStaticPathPattern(), (registration) -> {
 				registration.addResourceLocations(this.resourceProperties.getStaticLocations());
 				if (this.servletContext != null) {
@@ -351,7 +344,7 @@ public void addResourceHandlers(ResourceHandlerRegistry registry) {
 			});
 		}
 
-		private void addResourceHandler(ResourceHandlerRegistry registry, String pattern, Resource... locations) {
+		private void addResourceHandler(ResourceHandlerRegistry registry, String pattern, String... locations) {
 			addResourceHandler(registry, pattern, (registration) -> registration.addResourceLocations(locations));
 		}
 

spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/reactive/WebFluxAutoConfigurationTests.java

@@ -160,10 +160,9 @@ void shouldRegisterResourceHandlerMapping() {
 			SimpleUrlHandlerMapping hm = context.getBean("resourceHandlerMapping", SimpleUrlHandlerMapping.class);
 			assertThat(hm.getUrlMap().get("/**")).isInstanceOf(ResourceWebHandler.class);
 			ResourceWebHandler staticHandler = (ResourceWebHandler) hm.getUrlMap().get("/**");
-			assertThat(staticHandler).extracting("locationValues").asList().hasSize(2);
-			assertThat(staticHandler.getLocations()).hasSize(2);
-			assertThat(staticHandler.getLocations().get(0)).hasToString("class path resource [META-INF/resources/]");
-			assertThat(staticHandler.getLocations().get(1)).hasToString("class path resource [public/]");
+			assertThat(staticHandler).extracting("locationValues").asList().hasSize(4);
+			assertThat(staticHandler.getLocations()).hasSize(1);
+			assertThat(staticHandler.getLocations().get(0)).hasToString("class path resource [public/]");
 			assertThat(hm.getUrlMap().get("/webjars/**")).isInstanceOf(ResourceWebHandler.class);
 			ResourceWebHandler webjarsHandler = (ResourceWebHandler) hm.getUrlMap().get("/webjars/**");
 			assertThat(webjarsHandler).extracting("locationValues").asList()
@@ -177,7 +176,7 @@ void shouldMapResourcesToCustomPath() {
 			SimpleUrlHandlerMapping hm = context.getBean("resourceHandlerMapping", SimpleUrlHandlerMapping.class);
 			assertThat(hm.getUrlMap().get("/static/**")).isInstanceOf(ResourceWebHandler.class);
 			ResourceWebHandler staticHandler = (ResourceWebHandler) hm.getUrlMap().get("/static/**");
-			assertThat(staticHandler).extracting("locationValues").asList().hasSize(2);
+			assertThat(staticHandler).extracting("locationValues").asList().hasSize(4);
 		});
 	}
 

spring-boot-project/spring-boot-autoconfigure/src/test/resources/META-INF/resources/webjars/webjar

@@ -1642,7 +1642,7 @@ bom {
 			]
 		}
 	}
-	library("Spring Framework", "5.3.11") {
+	library("Spring Framework", "5.3.12-SNAPSHOT") {
 		group("org.springframework") {
 			imports = [
 				"spring-framework-bom"