In my client code, I want to be able to handle error situations like "channel not found", "bad credentials" etc.
In your current implementation, org.igniterealtime.restclient.RestClient.call(...), you only log error messages in the isStatusCodeOK(...) call, but you ultimately return null on any error. This prevents client code from handling errors accordingly.
Please either let JAXRS' ResponseProcessingException, ProcessingException and WebApplicationException through, or introduce business exceptions for all possible error cases.
Redor
commented
7 years ago
In my client code, I want to be able to handle error situations like "channel not found", "bad credentials" etc.
In your current implementation,
org.igniterealtime.restclient.RestClient.call(...), you only log error messages in theisStatusCodeOK(...)call, but you ultimately returnnullon any error. This prevents client code from handling errors accordingly.Please either let JAXRS'
ResponseProcessingException,ProcessingExceptionandWebApplicationExceptionthrough, or introduce business exceptions for all possible error cases.