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

Work on CAMEL-25902 started by Andrea Cosentino.
------------------------------------------------
> camel-netty - add a muteException consumer option
> -------------------------------------------------
>
>                 Key: CAMEL-25902
>                 URL: https://issues.apache.org/jira/browse/CAMEL-25902
>             Project: Camel
>          Issue Type: Improvement
>          Components: camel-netty
>            Reporter: Andrea Cosentino
>            Assignee: Andrea Cosentino
>            Priority: Major
>             Fix For: 4.18.6, 4.22.3, 4.24.0
>
>
> CAMEL-24476, CAMEL-24477, CAMEL-24478, CAMEL-25087 and CAMEL-25460 added a 
> {{muteException}} consumer option to camel-mina, camel-cxf, camel-grpc, 
> camel-coap and camel-thrift, defaulting to {{true}} in line with the http 
> family after CAMEL-23651. The plain camel-netty consumer (TCP/UDP) has no 
> such option.
> When the exchange failed and {{transferExchange}} is off, 
> {{ServerChannelHandler.getResponseBody}} uses the route's exception as the 
> reply:
> {code:java}
> boolean exception = exchange.getException() != null && 
> !consumer.getEndpoint().getConfiguration().isTransferExchange();
> if (exception) {
>     return exchange.getException();
> }
> {code}
> With {{textline=true}} the reply is the exception converted to a String, i.e. 
> its class name and message 
> ({{NettyTcpWithInOutUsingPlainSocketTest.testExchangeFailedOutShouldBeNull}} 
> asserts this). With an encoder that accepts any object, it is the serialised 
> exception.
> Proposal: add a {{muteException}} consumer option to {{NettyConfiguration}}, 
> defaulting to {{true}}, that replies with a generic exception carrying no 
> class name, message or stack trace (as camel-mina does), with an 
> upgrade-guide entry noting that routes relying on the detail must set it to 
> {{false}}. {{NettyHttpConfiguration}} already declares {{muteException}}; 
> move it up to {{NettyConfiguration}} rather than declaring it twice.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to