root
/
spring-framework
mirror of https://github.com/spring-projects/spring-framework.git
1
0
Fork 0
Code Issues Actions 4 Packages Projects Releases Wiki Activity

Restore log level for resolved exceptions 

The fix for SPR-17178 switched from debug to warn level warning for
all sub-classes of AbstractHandlerExceptionResolver where the request
concerned the DefaultHandlerExceptionResolver only.

This commit restores the original DEBUG level logging that was in
AbstractHandlerExceptionResolver from before SPR-17178. In addition
DefaultHandlerExceptionResolver registers a warnLogCategory by default
which enables warn logging and hence fulfilling the original goal
for SPR-17178.

Issue: SPR-17383
This commit is contained in:
Rossen Stoyanchev 2018-10-22 12:03:19 -04:00
parent de453ace86
commit 41e6aa6de2
3 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
spring-web/src/main/java/org/springframework/web/server/handler/ResponseStatusExceptionHandler.java
View File

@ -72,8 +72,8 @@ public class ResponseStatusExceptionHandler implements WebExceptionHandler {
if (this.warnLogger != null && this.warnLogger.isWarnEnabled()) { if (this.warnLogger != null && this.warnLogger.isWarnEnabled()) {
this.warnLogger.warn(logPrefix + formatError(ex, exchange.getRequest()), ex); this.warnLogger.warn(logPrefix + formatError(ex, exchange.getRequest()), ex);
} }
else if (logger.isWarnEnabled()) { else if (logger.isDebugEnabled()) {
logger.warn(logPrefix + formatError(ex, exchange.getRequest())); logger.debug(logPrefix + formatError(ex, exchange.getRequest()));
} }
return exchange.getResponse().setComplete(); return exchange.getResponse().setComplete();

4
spring-webmvc/src/main/java/org/springframework/web/servlet/handler/AbstractHandlerExceptionResolver.java
View File

@ -136,8 +136,8 @@ public abstract class AbstractHandlerExceptionResolver implements HandlerExcepti
ModelAndView result = doResolveException(request, response, handler, ex); ModelAndView result = doResolveException(request, response, handler, ex);
if (result != null) { if (result != null) {
// Print warn message when warn logger is not enabled... // Print warn message when warn logger is not enabled...
if (logger.isWarnEnabled() && (this.warnLogger == null || !this.warnLogger.isWarnEnabled())) { if (logger.isDebugEnabled() && (this.warnLogger == null || !this.warnLogger.isWarnEnabled())) {
logger.warn("Resolved [" + ex + "]" + (result.isEmpty() ? "" : " to " + result)); logger.debug("Resolved [" + ex + "]" + (result.isEmpty() ? "" : " to " + result));
} }
// warnLogger with full stack trace (requires explicit config) // warnLogger with full stack trace (requires explicit config)
logException(ex, request); logException(ex, request);

1
spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/support/DefaultHandlerExceptionResolver.java
View File

@ -159,6 +159,7 @@ public class DefaultHandlerExceptionResolver extends AbstractHandlerExceptionRes
*/ */
public DefaultHandlerExceptionResolver() { public DefaultHandlerExceptionResolver() {
setOrder(Ordered.LOWEST_PRECEDENCE); setOrder(Ordered.LOWEST_PRECEDENCE);
setWarnLogCategory(getClass().getName());
} }