Spring/spring-framework
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Minor refactoring in WebAsyncManager

Browse Source 
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
This commit is contained in:
rstoyanchev
2025-01-06 10:55:52 +00:00
parent 245341231f
commit c50cb10964
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
spring-web/src/main/java/org/springframework/web/context/request/async/WebAsyncManager.java
View File

@@ -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);