Spring Boot @ControllerAdvice Exception Handler not returning HTTP Status Text

Question

I have a GlobalExceptionHandler that catches exceptions and returns a HTTP Error codes.

@ControllerAdvice
@Component
public class GlobalExceptionHandler {

    private static final Logger logger = LoggerFactory.getLogger(GlobalExceptionHandler.class);

    ...

    // 404 - Not Found
    @ExceptionHandler(NoHandlerFoundException.class)
    @ResponseStatus(HttpStatus.NOT_FOUND)
    public void requestHandlingNoHandlerFound(HttpServletRequest request, Exception exception) {
        logger.error("Error Not Found (404): " + exception.getMessage());
    }

    ...

}

This works correctly and responds with a 404. But the HTTP response looks like this:

HTTP/1.1 404 
X-Application-Context: application:8080
Content-Length: 0
Date: Wed, 03 Aug 2016 14:36:52 GMT

But should return:

HTTP/1.1 404 Not Found
X-Application-Context: application:8080
Content-Length: 0
Date: Wed, 03 Aug 2016 14:36:52 GMT

The Not Found part is missing. This is the same for other errors. e.g. 500 - Internal Server Error

Any ideas in how to include this?

Update: Downgrading from Spring Boot 1.4.0 to 1.3.7 fixed this

I really, really hope you mean Spring Boot 1.4.0 -> 1.3.7

ipsi
– ipsi

2016-08-03 14:56:41 +00:00
Commented Aug 3, 2016 at 14:56 — ipsi
– ipsi, Commented Aug 3, 2016 at 14:56

Community · Accepted Answer · 2020-06-20 09:12:55Z

3

From the release notes:

Server header

The Server HTTP response header is no longer set unless the server.server-header property is set.

This seems like it might be the cause of your issue.

edited Jun 20, 2020 at 9:12

CommunityBot

11 silver badge

answered Aug 3, 2016 at 15:01

ipsi

2,07018 silver badges23 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

ptimson Over a year ago

Thanks, what would this need to be set to?

ipsi Over a year ago

Hah, my bad - applies to something completely different. This looks like a regression, and it would probably be worth reporting a bug for it.

Grigory Kislin Over a year ago

From tomcat.apache.org/tomcat-8.5-doc/changelog.html: RFC 7230 states that clients should ignore reason phrases in HTTP/1.1 response messages. Since the reason phrase is optional, Tomcat no longer sends it.

M. Deinum · Accepted Answer · 2016-10-14 05:46:06Z

1

@ControllerAdvice
public class RestResponseEntityExceptionHandler extends ResponseEntityExceptionHandler{

by extending this class you can override default spring behavior.

@ExceptionHandler({NoHandlerFoundException.class,EntityNotFoundException.class})
protected ResponseEntity<Object> handleNotFound(final RuntimeException ex,final WebRequest request) {
        final MyError myError= new MyError (HttpStatus.NOT_FOUND, ex);
        return handleExceptionInternal(ex, myError, new HttpHeaders(), HttpStatus.NOT_FOUND, request);
}

@ResponseStatus(HttpStatus.NOT_FOUND) is ok, but to get more control over response use ResponseEntity. Also by extending ResponseEntityExceptionHandler you get access to bunch of preconfigured exception handlers.

edited Oct 14, 2016 at 5:46

M. Deinum

127k22 gold badges235 silver badges253 bronze badges

answered Aug 4, 2016 at 10:22

SeaBiscuit

2,6214 gold badges27 silver badges44 bronze badges

1 Comment

ptimson Over a year ago

I'll give this a go. Thanks :)

Collectives™ on Stack Overflow

Spring Boot @ControllerAdvice Exception Handler not returning HTTP Status Text

2 Answers 2

Server header

3 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Server header

3 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related