DisconnectedClientHelper should only be used for logging

See spring-projects#34264 (comment)

Signed-off-by: Yanming Zhou <[email protected]>

Author: quaff

spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/support/DefaultHandlerExceptionResolver.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2002-2024 the original author or authors.
+ * Copyright 2002-2025 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -51,7 +51,6 @@
 import org.springframework.web.servlet.NoHandlerFoundException;
 import org.springframework.web.servlet.handler.AbstractHandlerExceptionResolver;
 import org.springframework.web.servlet.resource.NoResourceFoundException;
-import org.springframework.web.util.DisconnectedClientHelper;
 import org.springframework.web.util.WebUtils;
 
 /**
@@ -149,6 +148,7 @@
  * @author Arjen Poutsma
  * @author Rossen Stoyanchev
  * @author Juergen Hoeller
+ * @author Yanming Zhou
  * @since 3.0
  * @see org.springframework.web.servlet.mvc.method.annotation.ResponseEntityExceptionHandler
  */
@@ -246,9 +246,6 @@ else if (ex instanceof AsyncRequestNotUsableException) {
 				return handleAsyncRequestNotUsableException(
 						(AsyncRequestNotUsableException) ex, request, response, handler);
 			}
-			else if (DisconnectedClientHelper.isClientDisconnectedException(ex)) {
-				return handleDisconnectedClientException(ex, request, response, handler);
-			}
 		}
 		catch (Exception handlerEx) {
 			if (logger.isWarnEnabled()) {
@@ -505,26 +502,6 @@ protected ModelAndView handleAsyncRequestNotUsableException(AsyncRequestNotUsabl
 		return new ModelAndView();
 	}
 
-	/**
-	 * Handle an Exception that indicates the client has gone away. This is
-	 * typically an {@link IOException} of a specific subtype or with a message
-	 * specific to the underlying Servlet container. Those are detected through
-	 * {@link DisconnectedClientHelper#isClientDisconnectedException(Throwable)}
-	 * <p>By default, do nothing since the response is not usable.
-	 * @param ex the {@code Exception} to be handled
-	 * @param request current HTTP request
-	 * @param response current HTTP response
-	 * @param handler the executed handler, or {@code null} if none chosen
-	 * at the time of the exception (for example, if multipart resolution failed)
-	 * @return an empty ModelAndView indicating the exception was handled
-	 * @since 6.2
-	 */
-	protected ModelAndView handleDisconnectedClientException(
-			Exception ex, HttpServletRequest request, HttpServletResponse response, @Nullable Object handler) {
-
-		return new ModelAndView();
-	}
-
 	/**
 	 * Handle an {@link ErrorResponse} exception.
 	 * <p>The default implementation sets status and the headers of the response

spring-webmvc/src/test/java/org/springframework/web/servlet/mvc/support/DefaultHandlerExceptionResolverTests.java

@@ -251,13 +251,6 @@ void handleMaxUploadSizeExceededException() {
 		assertThat(response.getErrorMessage()).isEqualTo("Maximum upload size exceeded");
 	}
 
-	@Test
-	void handleClientDisconnectedException() {
-		SocketException ex = new SocketException("Connection reset");
-		ModelAndView mav = exceptionResolver.resolveException(request, response, null, ex);
-		assertThat(mav).as("No ModelAndView returned").isNotNull();
-	}
-
 	@Test
 	void handleRestClientExceptionHasConnectionResetMessage() {
 		RestClientException ex = new RestClientException("I/O error", new SocketException("Connection reset"));