From cc1514be8513e6c35813c94abbfd566571a16420 Mon Sep 17 00:00:00 2001 From: Brian Clozel Date: Tue, 10 Mar 2026 18:08:09 +0100 Subject: [PATCH] Fix DefaultResponseErrorHandler method visibility Prior to this commit, the `setMessageConverters` method would have private visibility. But `initBodyConvertFunction`, which is `protected`, relies on the message converters being set in the first place. While this works with `RestTemplate` because this is done automatically, the `RestClient` does offer a builder method to configure a `ResponseErrorHandler` and this makes it impossible to configure converters in this case. This commit aligns the method visibility by making it protected. Closes gh-36434 --- .../web/client/DefaultResponseErrorHandler.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/spring-web/src/main/java/org/springframework/web/client/DefaultResponseErrorHandler.java b/spring-web/src/main/java/org/springframework/web/client/DefaultResponseErrorHandler.java index 46768cba030..6e83480dd21 100644 --- a/spring-web/src/main/java/org/springframework/web/client/DefaultResponseErrorHandler.java +++ b/spring-web/src/main/java/org/springframework/web/client/DefaultResponseErrorHandler.java @@ -69,11 +69,10 @@ public class DefaultResponseErrorHandler implements ResponseErrorHandler { * to use to decode error content. * @since 6.0 */ - void setMessageConverters(List> converters) { + protected void setMessageConverters(List> converters) { this.messageConverters = Collections.unmodifiableList(converters); } - /** * Delegates to {@link #hasError(HttpStatusCode)} with the response status code. * @see ClientHttpResponse#getStatusCode()