You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@camel.apache.org by "Willem Jiang (JIRA)" <ji...@apache.org> on 2015/04/22 10:11:59 UTC

[jira] [Commented] (CAMEL-8674) Camel-Netty4 does not set remote UDP address in headers

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

Willem Jiang commented on CAMEL-8674:
-------------------------------------

Thanks for reporting this, I just wrote a simple test and produce the error.
Current we just set the CamelNettyRemoteAddress on the Exchange property to help us find a way to send the response message back.
I just updated the code of NettyEndpoint to setup the CamelNettyRemoteAddress on the message header at the same time.

> Camel-Netty4 does not set remote UDP address in headers
> -------------------------------------------------------
>
>                 Key: CAMEL-8674
>                 URL: https://issues.apache.org/jira/browse/CAMEL-8674
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-netty4
>    Affects Versions: 2.15.1
>            Reporter: Tuure Laurinolli
>            Assignee: Willem Jiang
>
> It appears that camel-netty4 does not set CamelNettyRemoteAddress in *headers* of an Exchange generated by an in-only UDP endpoint.
> It does set CamelNettyRemoteAddress in *properties* of the exchange.
> CamelNettyRemoteAddress in *properties* is set from AddressedEnvelope#sender(). CamelNettyRemoteAddress in *headers* is set from ChannelHandlerContext#channel().remoteAddress().
> It appears that the latter does not actually contain the remote address. Possible solutions are:
> 1) Change NettyEndpoint to set CamelNettyRemoteAddress in *headers* from the AddressedEnvelope received
> 2) Change NettyEndpoint to set CamelNettyRemoteAddress from *properties* (and to set *properties* of the Exchange first)



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