diff --git a/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/servlet/MultipartAutoConfiguration.java b/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/servlet/MultipartAutoConfiguration.java index 6cc7464821..d2b52049f4 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/servlet/MultipartAutoConfiguration.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/servlet/MultipartAutoConfiguration.java @@ -30,7 +30,6 @@ import org.springframework.boot.web.servlet.context.ServletWebServerApplicationC import org.springframework.context.annotation.Bean; import org.springframework.context.annotation.Configuration; import org.springframework.web.multipart.MultipartResolver; -import org.springframework.web.multipart.commons.CommonsMultipartResolver; import org.springframework.web.multipart.support.StandardServletMultipartResolver; import org.springframework.web.servlet.DispatcherServlet; @@ -63,7 +62,7 @@ public class MultipartAutoConfiguration { } @Bean - @ConditionalOnMissingBean({ MultipartConfigElement.class, CommonsMultipartResolver.class }) + @ConditionalOnMissingBean(MultipartConfigElement.class) public MultipartConfigElement multipartConfigElement() { return this.multipartProperties.createMultipartConfig(); } diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/servlet/MultipartAutoConfigurationTests.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/servlet/MultipartAutoConfigurationTests.java index 24180f7b1c..c93c676f67 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/servlet/MultipartAutoConfigurationTests.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/servlet/MultipartAutoConfigurationTests.java @@ -43,7 +43,6 @@ import org.springframework.web.bind.annotation.RequestMapping; import org.springframework.web.bind.annotation.ResponseBody; import org.springframework.web.client.RestTemplate; import org.springframework.web.multipart.MultipartResolver; -import org.springframework.web.multipart.commons.CommonsMultipartResolver; import org.springframework.web.multipart.support.StandardServletMultipartResolver; import org.springframework.web.servlet.DispatcherServlet; import org.springframework.web.servlet.config.annotation.EnableWebMvc; @@ -188,15 +187,6 @@ class MultipartAutoConfigurationTests { assertThat(this.context.getBeansOfType(MultipartConfigElement.class)).hasSize(1); } - @Test - void containerWithCommonsMultipartResolver() { - this.context = new AnnotationConfigServletWebServerApplicationContext( - ContainerWithCommonsMultipartResolver.class, BaseConfiguration.class); - MultipartResolver multipartResolver = this.context.getBean(MultipartResolver.class); - assertThat(multipartResolver).isInstanceOf(CommonsMultipartResolver.class); - assertThat(this.context.getBeansOfType(MultipartConfigElement.class)).hasSize(0); - } - @Test void configureResolveLazily() { this.context = new AnnotationConfigServletWebServerApplicationContext(); @@ -406,16 +396,6 @@ class MultipartAutoConfigurationTests { } - @Configuration(proxyBeanMethods = false) - static class ContainerWithCommonsMultipartResolver { - - @Bean - CommonsMultipartResolver multipartResolver() { - return mock(CommonsMultipartResolver.class); - } - - } - @Controller static class WebController {