camel-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Claus Ibsen (JIRA)" <j...@apache.org>
Subject [jira] Commented: (CAMEL-3209) CXF-RS - Add throwExceptionOnFailure option
Date Tue, 09 Nov 2010 07:44:00 GMT

    [ https://issues.apache.org/activemq/browse/CAMEL-3209?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=63191#action_63191
] 

Claus Ibsen commented on CAMEL-3209:
------------------------------------

Looks nice. Adding some unit test must be done :)

And there is one code comment which is off by one number
>  // Throw exception on non 201

It should be non 200 :)

I assume you have kept the option to false by default to keep backwards compatibility.
I would prefer it was true as default as an exception will be thrown and the error handling
of Camel can kick in.
This will also make it similar to the http components.

Any thoughts of changing it to true as default?


> CXF-RS - Add throwExceptionOnFailure option
> -------------------------------------------
>
>                 Key: CAMEL-3209
>                 URL: https://issues.apache.org/activemq/browse/CAMEL-3209
>             Project: Apache Camel
>          Issue Type: New Feature
>          Components: camel-cxf
>            Reporter: Claus Ibsen
>            Assignee: Johan Edstrom
>             Fix For: 2.6.0
>
>         Attachments: 0001-https-issues.apache.org-activemq-browse-CAMEL-3209.patch, 0002-https-issues.apache.org-activemq-browse-CAMEL-3209.patch,
Camel____CXF_jax-rs_producer_exception.patch
>
>
> See CAMEL-3184
> Currently you will have to check the status code of the response if its != 200.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message