CODEWITHSUNDEEP
javajerseyexceptionmapper
Seyed Ali Roshan
Seyed Ali Roshan

Reputation: 1574

JAX-RS (jersey implementation) ExceptionMapper<Throwable> aren't catching ConstraintViolationException

I have an ExceptionMapper as a @Provider for handling all of my exceptions.

So it is obvious that my class implements ExceptionMapper<Throwable> and we as know, all exceptions are extended Throwable somehow.

Now I used jax-rs @NotNull to checking my resources input values to be not null and by some search, I realized it will throw ConstraintViolationException when the annotated field is null.

So I tried to handle it and add some details to response (add some custom json model) in my ExceptionMapper like this:

@Provider
public class AllExceptionMapper implements ExceptionMapper<Throwable> {

    private final Logger logger = LogManager.getLogger(AllExceptionMapper.class);

    @Override
    public Response toResponse(Throwable ex) {
        Response response;
        CustomError error;
        // handle some errors

        else if (ex instanceof ConstraintViolationException) {
             error = new CustomError(
                 324, // predefined error type in our documents
                 "some details"
             );
             response = Response.status(Response.Status.BAD_REQUEST).entity(error).build();
        }

        // handle some other errors

        return response;
    }

}

The problem is this didn't work but if I would create another exception mapper provider which implements ExceptionMapper<ConstraintViolationException> and handle it there, it works without any problem.

As I said before (and also checked) all exceptions are extended from Throwable class somehow so what am I missing and why it is not working?

......

By didn't work, I mean it ignores my mapper (the one which implements from ExceptionMapper<Throwable>) and had the normal behavior which is returning the status code 400 with no response payload like there is no mapper at all

Upvotes: 2

Views: 2413

Answers (2)

Jim
Jim

Reputation: 71

Don't using @Privider and should be using 'ResourceConfig.register(MyExceptionMapper);'

 public class ErrorHandler implements ExceptionMapper<Throwable> {

    private final static Logger LOGGER = LoggerFactory.getLogger(ErrorHandler.class);

    @Override
    public Response toResponse(Throwable e) {

        LOGGER.error("Handle a unknown error, " + e.getClass() + ": " + e.getMessage(), e);
        e.printStackTrace();

        Response.ResponseBuilder responseBuilder = Response.status(status);
        responseBuilder.entity(exception);

        return responseBuilder.build();
    }
}


class Config extends ResourceConfig {

    private final static Logger LOGGER = LoggerFactory.getLogger(Main.class);

    public Config() {
        super();
        super.register(ErrorHandler.class);
    }
}

Upvotes: 0

SBylemans
SBylemans

Reputation: 1764

The way the ExceptionMapper is supposed to work, is that you can create a generic ExceptionMapper<Throwable>, which handles all errors. And then you can create more specific ExceptionMappers which handle the more specific errors in another way. All of this in separate classes.

The fact that doing it in a separate class works for you, lets me believe that there's a more specific ExceptionMapper somewhere, which handles the exception before you can.

The way ExceptionMappers are intended to be used is actually very clean, and also keeps your code clean. Wanting to keep the code in one central place, will result in a giant if...

Upvotes: 2

Related Questions