Spring MVC: использование @ResponseStatus (reason = '') в обработчике исключений @ResponseBody в tomcat - PullRequest
7 голосов
/ 12 апреля 2011

Кто-нибудь знает, почему я не могу использовать @ResponseStatus(reason = "My message") в обработчике исключений весной MVC, все еще возвращая @ResponseBody.Кажется, что происходит, если я использую атрибут reason

// this exception handle works, the result is a 404 and the http body is the json serialised
// {"message", "the message"}
@ExceptionHandler
@ResponseStatus(value = HttpStatus.NOT_FOUND)
public Map<String, String> notFoundHandler(NotFoundException e){
    return Collections.singletonMap("message", e.getMessage());
}

// this doesn't... the response is a 404 and the status line reads 'Really really not found'
// but the body is actually the standard Tomcat 404 page
@ExceptionHandler
@ResponseStatus(value = HttpStatus.NOT_FOUND, reason = "Really really not found")
public Map<String, String> reallyNotFoundHandler(ReallyNotFoundException e){
    return Collections.singletonMap("message", e.getMessage());
}

Код для этого примера на github заканчивается.

Ответы [ 2 ]

5 голосов
/ 14 апреля 2011

Кажется, что это прямой результат следующего кода из AnnotationMethodHandlerExceptionResolver

private ModelAndView getModelAndView(Method handlerMethod, Object returnValue, ServletWebRequest webRequest)
        throws Exception {

    ResponseStatus responseStatusAnn = AnnotationUtils.findAnnotation(handlerMethod, ResponseStatus.class);
    if (responseStatusAnn != null) {
        HttpStatus responseStatus = responseStatusAnn.value();
        String reason = responseStatusAnn.reason();
        if (!StringUtils.hasText(reason)) {
            // this doesn't commit the response
            webRequest.getResponse().setStatus(responseStatus.value());
        }
        else {
            // this commits the response such that any more calls to write to the 
            // response are ignored
            webRequest.getResponse().sendError(responseStatus.value(), reason);
        }
    }
    /// snip
}

Об этом Спрингсорсу сообщили в SPR-8251 :

3 голосов
/ 12 октября 2013

Для записи, начиная с весны 3.2, это стало еще хуже, потому что AnnotationMethodHandlerExceptionResolver был заменен на ResponseStatusExceptionResolver, и это делает:

protected ModelAndView resolveResponseStatus(ResponseStatus responseStatus, HttpServletRequest request,
  HttpServletResponse response, Object handler, Exception ex) throws Exception {

  int statusCode = responseStatus.value().value();
  String reason = responseStatus.reason();
  if (this.messageSource != null) {
    reason = this.messageSource.getMessage(reason, null, reason, LocaleContextHolder.getLocale());
  }
  if (!StringUtils.hasLength(reason)) {
    response.sendError(statusCode);
  }
  else {
    response.sendError(statusCode, reason);
  }
  return new ModelAndView();
}

Это стоит сообщения об ошибке. Кроме того, @ResponseStatus задокументировано с setStatus и является плохо разработанным. Он должен был называться @ResponseError.

Наконец-то я создал две проблемы: SPR-11192 и SPR-11193 .

Прошел почти год, и мои два вопроса все еще открыты. Я не считаю Spring WebMVC первоклассным фреймворком REST, который не является имхо, Web MVC предназначен для humas, а не для машин: - (

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...