From 3090a3a71f6bdee4418a0b25d9812bdc9ee53b87 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?F=C3=A1bio=20Ramon?= Date: Tue, 23 Mar 2021 23:50:37 -0300 Subject: [PATCH] Ignore quality value when removing MediaType.ALL Update the default reactive exception handler so that `MediaType.ALL` is removed regardless of any quality setting. Prior to this commit, the "match-all" media type was not properly ignored if it has a quality value and would show HTML content if the accept header was `application/json, */*;q=0.9`. See gh-25778 --- .../DefaultErrorWebExceptionHandler.java | 2 +- .../DefaultErrorWebExceptionHandlerTests.java | 21 +++++++++++++++++++ 2 files changed, 22 insertions(+), 1 deletion(-) diff --git a/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/reactive/error/DefaultErrorWebExceptionHandler.java b/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/reactive/error/DefaultErrorWebExceptionHandler.java index ea6cd7edf91..cc5fb2f7df1 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/reactive/error/DefaultErrorWebExceptionHandler.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/reactive/error/DefaultErrorWebExceptionHandler.java @@ -237,7 +237,7 @@ public class DefaultErrorWebExceptionHandler extends AbstractErrorWebExceptionHa return (serverRequest) -> { try { List acceptedMediaTypes = serverRequest.headers().accept(); - acceptedMediaTypes.remove(MediaType.ALL); + acceptedMediaTypes.removeIf((mediaType) -> MediaType.ALL.equalsTypeAndSubtype(mediaType)); MediaType.sortBySpecificityAndQuality(acceptedMediaTypes); return acceptedMediaTypes.stream().anyMatch(MediaType.TEXT_HTML::isCompatibleWith); } diff --git a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/reactive/error/DefaultErrorWebExceptionHandlerTests.java b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/reactive/error/DefaultErrorWebExceptionHandlerTests.java index 3ad649c1848..e52a29e9199 100644 --- a/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/reactive/error/DefaultErrorWebExceptionHandlerTests.java +++ b/spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/web/reactive/error/DefaultErrorWebExceptionHandlerTests.java @@ -17,6 +17,7 @@ package org.springframework.boot.autoconfigure.web.reactive.error; import java.util.Collections; +import java.util.List; import java.util.Map; import org.junit.jupiter.api.Test; @@ -28,9 +29,12 @@ import org.springframework.boot.web.reactive.context.AnnotationConfigReactiveWeb import org.springframework.boot.web.reactive.error.ErrorAttributes; import org.springframework.context.ApplicationContext; import org.springframework.http.MediaType; +import org.springframework.http.codec.HttpMessageReader; +import org.springframework.http.codec.ServerCodecConfigurer; import org.springframework.mock.http.server.reactive.MockServerHttpRequest; import org.springframework.mock.web.server.MockServerWebExchange; import org.springframework.test.util.ReflectionTestUtils; +import org.springframework.web.reactive.function.server.ServerRequest; import org.springframework.web.reactive.result.view.View; import org.springframework.web.reactive.result.view.ViewResolver; import org.springframework.web.server.ServerWebExchange; @@ -86,4 +90,21 @@ class DefaultErrorWebExceptionHandlerTests { exceptionHandler.setViewResolvers(Collections.singletonList(viewResolver)); } + @Test + void acceptsTextHtmlShouldNotConsiderMediaAllEvenWithQuality() { + ErrorAttributes errorAttributes = mock(ErrorAttributes.class); + ResourceProperties resourceProperties = new ResourceProperties(); + ErrorProperties errorProperties = new ErrorProperties(); + ApplicationContext context = new AnnotationConfigReactiveWebApplicationContext(); + DefaultErrorWebExceptionHandler exceptionHandler = new DefaultErrorWebExceptionHandler(errorAttributes, + resourceProperties, errorProperties, context); + MediaType allWithQuality = new MediaType(MediaType.ALL.getType(), MediaType.ALL.getSubtype(), 0.9); + MockServerWebExchange exchange = MockServerWebExchange + .from(MockServerHttpRequest.get("/test").accept(allWithQuality)); + List> readers = ServerCodecConfigurer.create().getReaders(); + ServerRequest request = ServerRequest.create(exchange, readers); + boolean accepts = exceptionHandler.acceptsTextHtml().test(request); + assertThat(accepts).isFalse(); + } + }