From c50cb10964cc4cf98b4ceb5da8b7c181b67955e6 Mon Sep 17 00:00:00 2001
From: rstoyanchev <rossen.stoyanchev@broadcom.com>
Date: Mon, 6 Jan 2025 10:55:52 +0000
Subject: [PATCH] Minor refactoring in WebAsyncManager

There is no need to set the DeferredResult from WebAsyncManager in an
onError notification because it is already done from the Lifecycle
interceptor in DeferredResult.

See gh-34192
---
 .../web/context/request/async/WebAsyncManager.java           | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/spring-web/src/main/java/org/springframework/web/context/request/async/WebAsyncManager.java b/spring-web/src/main/java/org/springframework/web/context/request/async/WebAsyncManager.java
index e42cdb17cb..10e98d38d3 100644
--- a/spring-web/src/main/java/org/springframework/web/context/request/async/WebAsyncManager.java
+++ b/spring-web/src/main/java/org/springframework/web/context/request/async/WebAsyncManager.java
@@ -436,10 +436,7 @@ public final class WebAsyncManager {
 				logger.debug("Servlet container error notification for " + formatUri(this.asyncWebRequest));
 			}
 			try {
-				if (!interceptorChain.triggerAfterError(this.asyncWebRequest, deferredResult, ex)) {
-					return;
-				}
-				deferredResult.setErrorResult(ex);
+				interceptorChain.triggerAfterError(this.asyncWebRequest, deferredResult, ex);
 			}
 			catch (Throwable interceptorEx) {
 				setConcurrentResultAndDispatch(interceptorEx);