healchow commented on code in PR #5676:
URL: https://github.com/apache/inlong/pull/5676#discussion_r954445266
##########
inlong-manager/manager-web/src/main/java/org/apache/inlong/manager/web/config/ControllerExceptionHandler.java:
##########
@@ -45,14 +46,15 @@
@RestControllerAdvice
public class ControllerExceptionHandler {
- private static final String ERROR_MSG = "failed to handle request on path:
%s by user: %s";
+ private static final String ERROR_MSG = "failed to handle request on path:
%s by user: %s\n%s\n%s";
@ExceptionHandler(ConstraintViolationException.class)
public Response<String>
handleConstraintViolationException(HttpServletRequest request,
ConstraintViolationException e) {
UserInfo userInfo = LoginUserUtils.getLoginUser();
String username = userInfo != null ? userInfo.getName() : "";
- log.error(String.format(ERROR_MSG, request.getRequestURI(), username),
e);
+ log.error(String.format(ERROR_MSG, request.getRequestURI(), username,
e.getMessage(),
Review Comment:
In log info, the `getMessage` is not necessary when printing all stack
traces.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@inlong.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
