2024-01-02T12:14:33.118Z

1

답변 감사합니다. 김영한님 강의 정말 잘 듣고있습니다.

UserHandlerExceptionResolver의 resolveException()은 if문에 의해

accept헤더가 application/json인 경우와

TEXT/HTML인 경우 모두를 처리하는 코드라고

생각했습니다!

질문 드렸던 부분은 else이후에 나오는

accept헤더가 json이 아닌 경우에도

userException이 발생한경우 if분기문 이전에 상태코드를 400으로 바꾸었으므로

500에러 페이지대신 400을 뿌리도록 해야하는게 원래는 맞지 않을까 라는 질문이었습니다.

@Override
public ModelAndView resolveException(HttpServletRequest request, HttpServletResponse response, Object handler,
                                     Exception ex) {
    try {

        if (ex instanceof UserException) {
            log.info("userException resolver to 400");
            String acceptHeader = request.getHeader("accept");
            response.setStatus(HttpServletResponse.SC_BAD_REQUEST);

            if ("application/json".equals(acceptHeader)) {
                Map<String, Object> errorResult = new HashMap<>();
                errorResult.put("ex", ex.getClass());
                errorResult.put("message", ex.getMessage());
                String result = objectMapper.writeValueAsString(errorResult);

                response.setContentType("application/json");
                response.setCharacterEncoding("utf-8");
                response.getWriter().write(result);
                return new ModelAndView();

            } else {
                // TEXT/HTML
                return new ModelAndView("error/500");  // <<질문드렸던 부분
            }
        }
    } catch (IOException e) {
        log.error("resolver ex", e);
    }
    return null;
}

2

오~ 김허수님 감사합니다.

이 부분은 메뉴얼을 수정해야겠네요 🙂

그냥 넘어갈 수 도 있는데, 이렇게 말씀해주셔서 다른 분들께도 도움이 될 것 같아요. 진심으로 감사합니다^^

다음 버전에 패치할께요 🙂

2024-01-01T16:26:20.980Z