Skip to content

Commit cdc0247

Browse files
ServiceExceptionMapper log messages use placeholders (#3102)
1 parent 947b7f1 commit cdc0247

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

conjure-java-jersey-server/src/main/java/com/palantir/conjure/java/server/jersey/ServiceExceptionMapper.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,15 +42,15 @@ public Response toResponseInner(ServiceException exception) {
4242
int httpStatus = exception.getErrorType().httpErrorCode();
4343
if (httpStatus / 100 == 4 /* client error */) {
4444
log.info(
45-
"Error handling request",
46-
SafeArg.of("errorInstanceId", exception.getErrorInstanceId()),
45+
"Error handling request. {}: {}",
4746
SafeArg.of("errorName", exception.getErrorType().name()),
47+
SafeArg.of("errorInstanceId", exception.getErrorInstanceId()),
4848
exception);
4949
} else {
5050
log.error(
51-
"Error handling request",
52-
SafeArg.of("errorInstanceId", exception.getErrorInstanceId()),
51+
"Error handling request. {}: {}",
5352
SafeArg.of("errorName", exception.getErrorType().name()),
53+
SafeArg.of("errorInstanceId", exception.getErrorInstanceId()),
5454
exception);
5555
}
5656

0 commit comments

Comments
 (0)