사소하지만 다음과 같이 되는게 맞을까요?

스프링 MVC 2편 - 백엔드 웹 개발 활용 기술

HandlerExceptionResolver 활용

해결된 질문

작성

361

예외를 통해 에러코드 500이 만들어 진것을

수정하여 400으로 고치는 로직이므로

error/400페이지를 보여주는게 맞는 흐름일 것 같은데

아닐까요?

// TEXT/HTML
 return new ModelAndView("error/400");

spring mvc

답변 2

김영한

지식공유자

안녕하세요. 김허수님

UserHandlerExceptionResolver 코드로 질문하신 것으로 이해했습니다.

여기서는 application/json으로 요청한 경우에 한정해서 예외를 처리하고 있습니다.

따라서 일반적인 HTML View를 반환하는 것이 아니라 response에 직접 응답값을 JSON 형식으로 바로 반환하게 됩니다. 따라서 view가 필요하지 않으므로 return new ModelAndView();와 같이 비워둡니다.

감사합니다.

김허수

질문자

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

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;
}