Skip to content

Commit 52479ee

Browse files
committed
Merge pull request #17863 from peterjurkovic
* gh-17863: Polish "Make logError() in AbstractErrorWebExceptionHandler overrideable" Make logError() in AbstractErrorWebExceptionHandler overrideable Closes gh-17863
2 parents e8de5a6 + 6328f2e commit 52479ee

File tree

1 file changed

+10
-1
lines changed

1 file changed

+10
-1
lines changed

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/reactive/error/AbstractErrorWebExceptionHandler.java

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -273,7 +273,16 @@ private boolean isDisconnectedClientErrorMessage(String message) {
273273
return (message.contains("broken pipe") || message.contains("connection reset by peer"));
274274
}
275275

276-
private void logError(ServerRequest request, ServerResponse response, Throwable throwable) {
276+
/**
277+
* Logs the {@code throwable} error for the given {@code request} and {@code response}
278+
* exchange. The default implementation logs all errors at debug level. Additionally,
279+
* any internal server error (500) is logged at error level.
280+
* @param request the request that was being handled
281+
* @param response the response that was being sent
282+
* @param throwable the error to be logged
283+
* @since 2.2.0
284+
*/
285+
protected void logError(ServerRequest request, ServerResponse response, Throwable throwable) {
277286
if (logger.isDebugEnabled()) {
278287
logger.debug(request.exchange().getLogPrefix() + formatError(throwable, request));
279288
}

0 commit comments

Comments
 (0)