[jira] [Commented] (CAMEL-16129) Avoid property binding via reflection in NettyConfiguration

["Claus Ibsen (Jira)" <ji...@apache.org>]

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

Claus Ibsen commented on CAMEL-16129:
-------------------------------------

Yes this is wrong, most of the options can be set automatic. Just that logic about port tcp/udp and a few others. But that should be in the component.

> Avoid property binding via reflection in NettyConfiguration 
> ------------------------------------------------------------
>
>                 Key: CAMEL-16129
>                 URL: https://issues.apache.org/jira/browse/CAMEL-16129
>             Project: Camel
>          Issue Type: Improvement
>          Components: camel-netty
>    Affects Versions: 3.7.1
>            Reporter: James Netherton
>            Priority: Major
>             Fix For: 3.7.2, 3.8.0
>
>
> Not too sure about the logic in NettyConfiguration.parseURI but it invokes PropertyBindingSupport.bindProperties. Depending on the combination of URI options it can lead to some of the NettyConfiguration setters being invoked via reflection.
> E.g I was testing this URI in camel-quarkus. Both sync & allowDefaultCodec are being set through reflection.
> netty:udp://localhost:8081?sync=false&allowDefaultCodec=false



--
This message was sent by Atlassian Jira
(v8.3.4#803005)