Skip to content

Commit c50cb10

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 2453412 commit c50cb10

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
@@ -436,10 +436,7 @@ public void startDeferredResultProcessing(
436436
logger.debug("Servlet container error notification for " + formatUri(this.asyncWebRequest));
437437
}
438438
try {
439-
if (!interceptorChain.triggerAfterError(this.asyncWebRequest, deferredResult, ex)) {
440-
return;
441-
}
442-
deferredResult.setErrorResult(ex);
439+
interceptorChain.triggerAfterError(this.asyncWebRequest, deferredResult, ex);
443440
}
444441
catch (Throwable interceptorEx) {
445442
setConcurrentResultAndDispatch(interceptorEx);

0 commit comments

Comments
 (0)