Skip to content

Commit d94e04d

Browse files
committed
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
1 parent 5a44897 commit d94e04d

File tree

1 file changed

+1
-4
lines changed

1 file changed

+1
-4
lines changed

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

+1-4
Original file line numberDiff line numberDiff line change
@@ -438,10 +438,7 @@ public void startDeferredResultProcessing(
438438
logger.debug("Servlet container error notification for " + formatUri(this.asyncWebRequest));
439439
}
440440
try {
441-
if (!interceptorChain.triggerAfterError(this.asyncWebRequest, deferredResult, ex)) {
442-
return;
443-
}
444-
deferredResult.setErrorResult(ex);
441+
interceptorChain.triggerAfterError(this.asyncWebRequest, deferredResult, ex);
445442
}
446443
catch (Throwable interceptorEx) {
447444
setConcurrentResultAndDispatch(interceptorEx);

0 commit comments

Comments
 (0)