Merge branch '2.3.x'

Closes spring-projectsgh-23205
dreis2211 · Sep 7, 2020 · 9bf6e1c · 9bf6e1c
2 parents 3dfe11e + 116b247
commit 9bf6e1c
Show file tree

Hide file tree

Showing 3 changed files with 29 additions and 3 deletions.
diff --git a/...ring-boot/src/main/java/org/springframework/boot/web/servlet/support/ErrorPageFilter.java b/...ring-boot/src/main/java/org/springframework/boot/web/servlet/support/ErrorPageFilter.java
@@ -43,7 +43,6 @@
 import org.springframework.boot.web.server.ErrorPageRegistrar;
 import org.springframework.boot.web.server.ErrorPageRegistry;
 import org.springframework.core.Ordered;
-import org.springframework.core.annotation.Order;
 import org.springframework.util.ClassUtils;
 import org.springframework.web.filter.OncePerRequestFilter;
 import org.springframework.web.util.NestedServletException;
@@ -62,8 +61,7 @@
  * @author Andy Wilkinson
  * @since 2.0.0
  */
-@Order(Ordered.HIGHEST_PRECEDENCE + 1)
-public class ErrorPageFilter implements Filter, ErrorPageRegistry {
+public class ErrorPageFilter implements Filter, ErrorPageRegistry, Ordered {
 
 	private static final Log logger = LogFactory.getLog(ErrorPageFilter.class);
 
@@ -298,6 +296,11 @@ else if (errorPage.getStatus() != null) {
 	public void destroy() {
 	}
 
+	@Override
+	public int getOrder() {
+		return Ordered.HIGHEST_PRECEDENCE + 1;
+	}
+
 	private static void addClassIfPresent(Collection<Class<?>> collection, String className) {
 		try {
 			collection.add(ClassUtils.forName(className, null));

diff --git a/.../main/java/org/springframework/boot/web/servlet/support/ErrorPageFilterConfiguration.java b/.../main/java/org/springframework/boot/web/servlet/support/ErrorPageFilterConfiguration.java
@@ -38,6 +38,7 @@ ErrorPageFilter errorPageFilter() {
 	@Bean
 	FilterRegistrationBean<ErrorPageFilter> errorPageFilterRegistration(ErrorPageFilter filter) {
 		FilterRegistrationBean<ErrorPageFilter> registration = new FilterRegistrationBean<>(filter);
+		registration.setOrder(filter.getOrder());
 		registration.setDispatcherTypes(DispatcherType.REQUEST, DispatcherType.ASYNC);
 		return registration;
 	}

diff --git a/.../java/org/springframework/boot/web/servlet/support/SpringBootServletInitializerTests.java b/.../java/org/springframework/boot/web/servlet/support/SpringBootServletInitializerTests.java
@@ -47,6 +47,7 @@
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
 import org.springframework.context.support.AbstractApplicationContext;
+import org.springframework.core.Ordered;
 import org.springframework.core.env.PropertySource;
 import org.springframework.mock.web.MockServletContext;
 import org.springframework.web.context.WebApplicationContext;
@@ -133,6 +134,27 @@ void errorPageFilterRegistrationCanBeDisabled() {
 		}
 	}
 
+	@Test
+	@SuppressWarnings("rawtypes")
+	void errorPageFilterIsRegisteredWithNearHighestPrecedence() {
+		WebServer webServer = new UndertowServletWebServerFactory(0).getWebServer((servletContext) -> {
+			try (AbstractApplicationContext context = (AbstractApplicationContext) new WithErrorPageFilter()
+					.createRootApplicationContext(servletContext)) {
+				Map<String, FilterRegistrationBean> registrations = context
+						.getBeansOfType(FilterRegistrationBean.class);
+				assertThat(registrations).hasSize(1);
+				FilterRegistrationBean errorPageFilterRegistration = registrations.get("errorPageFilterRegistration");
+				assertThat(errorPageFilterRegistration.getOrder()).isEqualTo(Ordered.HIGHEST_PRECEDENCE + 1);
+			}
+		});
+		try {
+			webServer.start();
+		}
+		finally {
+			webServer.stop();
+		}
+	}
+
 	@Test
 	@SuppressWarnings("rawtypes")
 	void errorPageFilterIsRegisteredForRequestAndAsyncDispatch() {