You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cxf.apache.org by "Sergey Beryozkin (JIRA)" <ji...@apache.org> on 2016/07/31 12:26:20 UTC

[jira] [Resolved] (CXF-6983) org.apache.cxf.jaxrs.provider.AbstractJAXBProvider#handleJAXBException doubles the error text for unmarshalling errors

     [ https://issues.apache.org/jira/browse/CXF-6983?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sergey Beryozkin resolved CXF-6983.
-----------------------------------
       Resolution: Fixed
    Fix Version/s: 3.1.8
                   3.2.0

> org.apache.cxf.jaxrs.provider.AbstractJAXBProvider#handleJAXBException doubles the error text for unmarshalling errors
> ----------------------------------------------------------------------------------------------------------------------
>
>                 Key: CXF-6983
>                 URL: https://issues.apache.org/jira/browse/CXF-6983
>             Project: CXF
>          Issue Type: Improvement
>          Components: JAX-RS
>    Affects Versions: 3.1.6
>         Environment: CXF 3.1.6 with JAXB annotated beans
>            Reporter: Mike Golod
>            Priority: Trivial
>             Fix For: 3.2.0, 3.1.8
>
>
> When unmarshal json payload to JAXB annotated bean sometimes we can get an UnmarshallingException (e.g. for dates). That exception already contains very descriptive message. AbstractJAXBProvider#handleExceptionStart extracts it. Then in AbstractJAXBProvider#handleJAXBException:
> {code:title=AbstractJAXBProvider#handleJAXBException}
> if (linked != null && linked.getMessage() != null) {
>             Throwable cause = linked;
>             while (read && cause != null) {
>                 if (cause instanceof XMLStreamException && cause.getMessage().startsWith("Maximum Number")) {
>                     throw ExceptionUtils.toWebApplicationException(null, JAXRSUtils.toResponse(413)); 
>                 }
>                 if (cause instanceof DepthExceededStaxException) {
>                     throw ExceptionUtils.toWebApplicationException(null, JAXRSUtils.toResponse(413)); 
>                 }
>                 cause = cause.getCause();
>             }
>             sb.append(linked.getMessage()).append(". "); // this causes the problem
>         }
> {code}
> This code adds another message from linked exception. Message text is the same. Finally we get something like:
> {quote}
> JAXBException occured: <error text>. <error text>.
> {quote}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)