Merge pull request #17863 from peterjurkovic

* gh-17863:
  Polish "Make logError() in AbstractErrorWebExceptionHandler overrideable"
  Make logError() in AbstractErrorWebExceptionHandler overrideable

Closes gh-17863
This commit is contained in:
Andy Wilkinson
2019-09-05 21:18:34 +01:00

View File

@@ -273,7 +273,16 @@ public abstract class AbstractErrorWebExceptionHandler implements ErrorWebExcept
return (message.contains("broken pipe") || message.contains("connection reset by peer"));
}
private void logError(ServerRequest request, ServerResponse response, Throwable throwable) {
/**
* Logs the {@code throwable} error for the given {@code request} and {@code response}
* exchange. The default implementation logs all errors at debug level. Additionally,
* any internal server error (500) is logged at error level.
* @param request the request that was being handled
* @param response the response that was being sent
* @param throwable the error to be logged
* @since 2.2.0
*/
protected void logError(ServerRequest request, ServerResponse response, Throwable throwable) {
if (logger.isDebugEnabled()) {
logger.debug(request.exchange().getLogPrefix() + formatError(throwable, request));
}