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

Improve logging in ReactiveTypeHandler

Browse Source 
See gh-34188
This commit is contained in:
rstoyanchev
2025-01-06 12:13:29 +00:00
parent 9eefdb8041
commit e1b06ccfaa
1 changed files with 24 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/ReactiveTypeHandler.java
View File

@@ -337,7 +337,14 @@ class ReactiveTypeHandler {
logger.debug("Send for " + this.emitter + " failed: " + ex); logger.debug("Send for " + this.emitter + " failed: " + ex);
} }
terminate(); terminate();
this.emitter.completeWithError(ex); try {
this.emitter.completeWithError(ex);
}
catch (Exception ex2) {
if (logger.isDebugEnabled()) {
logger.debug("Failure from emitter completeWithError: " + ex2);
}
}
return; return;
} }
} }
@@ -350,13 +357,27 @@ class ReactiveTypeHandler {
if (logger.isDebugEnabled()) { if (logger.isDebugEnabled()) {
logger.debug("Publisher for " + this.emitter + " failed: " + ex); logger.debug("Publisher for " + this.emitter + " failed: " + ex);
} }
this.emitter.completeWithError(ex); try {
this.emitter.completeWithError(ex);
}
catch (Exception ex2) {
if (logger.isDebugEnabled()) {
logger.debug("Failure from emitter completeWithError: " + ex2);
}
}
} }
else { else {
if (logger.isTraceEnabled()) { if (logger.isTraceEnabled()) {
logger.trace("Publisher for " + this.emitter + " completed"); logger.trace("Publisher for " + this.emitter + " completed");
} }
this.emitter.complete(); try {
this.emitter.complete();
}
catch (Exception ex2) {
if (logger.isDebugEnabled()) {
logger.debug("Failure from emitter complete: " + ex2);
}
}
} }
return; return;
} }