Skip to content

Commit 7ea612b

Browse files
peterjurkovicwilkinsona
authored andcommitted
Make logError() in AbstractErrorWebExceptionHandler overrideable
See gh-17863
1 parent e8de5a6 commit 7ea612b

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

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

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -263,17 +263,7 @@ public Mono<Void> handle(ServerWebExchange exchange, Throwable throwable) {
263263
.flatMap((response) -> write(exchange, response));
264264
}
265265

266-
private boolean isDisconnectedClientError(Throwable ex) {
267-
return DISCONNECTED_CLIENT_EXCEPTIONS.contains(ex.getClass().getSimpleName())
268-
|| isDisconnectedClientErrorMessage(NestedExceptionUtils.getMostSpecificCause(ex).getMessage());
269-
}
270-
271-
private boolean isDisconnectedClientErrorMessage(String message) {
272-
message = (message != null) ? message.toLowerCase() : "";
273-
return (message.contains("broken pipe") || message.contains("connection reset by peer"));
274-
}
275-
276-
private void logError(ServerRequest request, ServerResponse response, Throwable throwable) {
266+
protected void logError(ServerRequest request, ServerResponse response, Throwable throwable) {
277267
if (logger.isDebugEnabled()) {
278268
logger.debug(request.exchange().getLogPrefix() + formatError(throwable, request));
279269
}
@@ -284,6 +274,16 @@ private void logError(ServerRequest request, ServerResponse response, Throwable
284274
}
285275
}
286276

277+
private boolean isDisconnectedClientError(Throwable ex) {
278+
return DISCONNECTED_CLIENT_EXCEPTIONS.contains(ex.getClass().getSimpleName())
279+
|| isDisconnectedClientErrorMessage(NestedExceptionUtils.getMostSpecificCause(ex).getMessage());
280+
}
281+
282+
private boolean isDisconnectedClientErrorMessage(String message) {
283+
message = (message != null) ? message.toLowerCase() : "";
284+
return (message.contains("broken pipe") || message.contains("connection reset by peer"));
285+
}
286+
287287
private String formatError(Throwable ex, ServerRequest request) {
288288
String reason = ex.getClass().getSimpleName() + ": " + ex.getMessage();
289289
return "Resolved [" + reason + "] for HTTP " + request.methodName() + " " + request.path();

0 commit comments

Comments
 (0)