[
https://issues.apache.org/jira/browse/CXF-4912?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Parwiz Rezai updated CXF-4912:
------------------------------
Description:
I have two differnt exceptions:
{code:borderStyle=solid}
public class Exception1 extends Exception {
// blah blah
}
public class Exception2 extends Exception {
// blah blah
}
{code}
{code:title=Exception1ResponseMapper.java:borderStyle=solid}
import com.fasterxml.jackson.core.JsonParser;
import com.fasterxml.jackson.databind.MappingJsonFactory;
import java.io.InputStream;
import javax.ws.rs.core.Response;
import org.apache.cxf.jaxrs.client.ResponseExceptionMapper;
public class Exception1ResponseMapper implements
ResponseExceptionMapper<Exception1> {
@Override
public Exception1 fromResponse(Response response) {
try {
MappingJsonFactory factory = new MappingJsonFactory();
JsonParser parser = factory.createJsonParser((InputStream)
response.getEntity());
Exception1 exception = parser.readValueAs(Exception1.class);
return exception;
} catch (Exception ex) {
return new Exception1("Could not deserialize server side exception:
" + ex.getMessage());
}
}
{code}
{code:title=Exception2ResponseMapper.java:borderStyle=solid}
import com.fasterxml.jackson.core.JsonParser;
import com.fasterxml.jackson.databind.MappingJsonFactory;
import java.io.InputStream;
import javax.ws.rs.core.Response;
import org.apache.cxf.jaxrs.client.ResponseExceptionMapper;
public class Exception2ResponseMapper implements
ResponseExceptionMapper<Exception2> {
@Override
public Exception2 fromResponse(Response response) {
try {
MappingJsonFactory factory = new MappingJsonFactory();
JsonParser parser = factory.createJsonParser((InputStream)
response.getEntity());
Exception2 exception = parser.readValueAs(Exception2.class);
// do some specific work for exception 2 only
return exception;
} catch (Exception ex) {
return new Exception2("Could not deserialize server side exception:
" + ex.getMessage());
}
}
{code}
so one mapper does something different than the other one.
now my service method:
{code:borderStyle=solid}
public interface MyService {
SomeCoolObject myMethod() throws Exception2, Exception1;
}
public class MyServiceImpl implements MyService {
public SomeCoolObject myMethod() throws Exception2, Exception1 {
throw new Exception1("hey this exception will still go exception 2
mapper.. why?");
}
}
{code}
The selection for response mapper will always pick the first
exception listed and invoke the mapper for that guy.. it will never
invoke Exception1ResponseMapper even though our exception is of type
Exception1..
I think the code needs to lookup the mapper based on type as well
instead of generic first mapper found for that service.
was:
I have two differnt exceptions:
{code:borderStyle=solid}
public class Exception1 extends Exception {
// blah blah
}
public class Exception2 extends Exception {
// blah blah
}
public class Exception1ResponseMapper implements
ResponseExceptionMapper<Exception1> {
}
public class Exception2ResponseMapper implements
ResponseExceptionMapper<Exception2> {
}
{code}
so one mapper does something different than the other one.
now my service method:
{code:borderStyle=solid}
public interface MyService {
SomeCoolObject myMethod() throws Exception2, Exception1;
}
public class MyServiceImpl implements MyService {
public SomeCoolObject myMethod() throws Exception2, Exception1 {
throw new Exception1("hey this exception will still go exception 2
mapper.. why?");
}
}
{code}
The selection for response mapper will always pick the first
exception listed and invoke the mapper for that guy.. it will never
invoke Exception1ResponseMapper even though our exception is of type
Exception1..
I think the code needs to lookup the mapper based on type as well
instead of generic first mapper found for that service.
> cxf rest client always picks the first ResponseExceptionMapper for a method
> with different exception throws
> -----------------------------------------------------------------------------------------------------------
>
> Key: CXF-4912
> URL: https://issues.apache.org/jira/browse/CXF-4912
> Project: CXF
> Issue Type: Bug
> Components: JAX-RS
> Affects Versions: 2.7.3
> Environment: tomcat 6.33, java 1.6_34, spring 3.2.1.RELEASE.
> Reporter: Parwiz Rezai
>
> I have two differnt exceptions:
> {code:borderStyle=solid}
> public class Exception1 extends Exception {
> // blah blah
> }
> public class Exception2 extends Exception {
> // blah blah
> }
> {code}
> {code:title=Exception1ResponseMapper.java:borderStyle=solid}
> import com.fasterxml.jackson.core.JsonParser;
> import com.fasterxml.jackson.databind.MappingJsonFactory;
> import java.io.InputStream;
> import javax.ws.rs.core.Response;
> import org.apache.cxf.jaxrs.client.ResponseExceptionMapper;
> public class Exception1ResponseMapper implements
> ResponseExceptionMapper<Exception1> {
> @Override
> public Exception1 fromResponse(Response response) {
> try {
> MappingJsonFactory factory = new MappingJsonFactory();
> JsonParser parser = factory.createJsonParser((InputStream)
> response.getEntity());
> Exception1 exception = parser.readValueAs(Exception1.class);
> return exception;
> } catch (Exception ex) {
> return new Exception1("Could not deserialize server side exception:
> " + ex.getMessage());
> }
> }
> {code}
> {code:title=Exception2ResponseMapper.java:borderStyle=solid}
> import com.fasterxml.jackson.core.JsonParser;
> import com.fasterxml.jackson.databind.MappingJsonFactory;
> import java.io.InputStream;
> import javax.ws.rs.core.Response;
> import org.apache.cxf.jaxrs.client.ResponseExceptionMapper;
> public class Exception2ResponseMapper implements
> ResponseExceptionMapper<Exception2> {
> @Override
> public Exception2 fromResponse(Response response) {
> try {
> MappingJsonFactory factory = new MappingJsonFactory();
> JsonParser parser = factory.createJsonParser((InputStream)
> response.getEntity());
> Exception2 exception = parser.readValueAs(Exception2.class);
> // do some specific work for exception 2 only
> return exception;
> } catch (Exception ex) {
> return new Exception2("Could not deserialize server side exception:
> " + ex.getMessage());
> }
> }
> {code}
> so one mapper does something different than the other one.
> now my service method:
> {code:borderStyle=solid}
> public interface MyService {
> SomeCoolObject myMethod() throws Exception2, Exception1;
> }
> public class MyServiceImpl implements MyService {
> public SomeCoolObject myMethod() throws Exception2, Exception1 {
> throw new Exception1("hey this exception will still go exception 2
> mapper.. why?");
> }
> }
> {code}
> The selection for response mapper will always pick the first
> exception listed and invoke the mapper for that guy.. it will never
> invoke Exception1ResponseMapper even though our exception is of type
> Exception1..
> I think the code needs to lookup the mapper based on type as well
> instead of generic first mapper found for that service.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
