From d94e04d97afaa745bcbd1d7ec5609d5256517255 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 e27899fb25..beed2f5976 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
@@ -438,10 +438,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);