[ 
https://issues.apache.org/jira/browse/CXF-7989?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16794546#comment-16794546
 ] 

Andriy Redko commented on CXF-7989:
-----------------------------------

I think the rx() impl is fine (CompletionStageRxInvokerImpl to be precise), 
since it returns CompletableFuture, and the only exceptions the  
CompletableFuture returns are CompletionException / CancellationException. The 
JAXRSInvoker should properly pick up the cause of the CompletionException and 
pass it to exception mappers chain. Make sense?

> org.apache.cxf.jaxrs.JAXRSInvoker#checkFutureResponse should probably handle 
> CompletionException
> ------------------------------------------------------------------------------------------------
>
>                 Key: CXF-7989
>                 URL: https://issues.apache.org/jira/browse/CXF-7989
>             Project: CXF
>          Issue Type: Improvement
>          Components: JAX-RS
>    Affects Versions: 3.3.0
>            Reporter: Romain Manni-Bucau
>            Assignee: Andriy Redko
>            Priority: Major
>
> The check should probably unwrap CompletionException otherwise it can require 
> some more custom ExceptionMappers to achieve a correct behavior.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to