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

ASF subversion and git services commented on TS-4070:
-----------------------------------------------------

Commit 894ef7f859772508331bdb610036baa9fafe81c6 in trafficserver's branch 
refs/heads/master from [~cschombu]
[ https://git-wip-us.apache.org/repos/asf?p=trafficserver.git;h=894ef7f ]

TS-4070: RemapProcessor Forward Mapping w/ Recv Port failing w/ TS-2157 changes.
This closes #375.

During the rework of RemapProcessor.cc, RemapProcessor::setup_for_remap()
as part of the TS-2157 changeset, the port access API appears to have been
incorrectly modified to use the client_info.src_addr.host_order_port() API
[source port, host order] instead of the client_info.dst_addr.port()
[destination/receive port, network order] API. This caused port based
remapping based on the receive port to fail with ATS 6.0.0.


> RemapProcessor Forward Mapping with Recv Port failing with TS-2157 changes
> --------------------------------------------------------------------------
>
>                 Key: TS-4070
>                 URL: https://issues.apache.org/jira/browse/TS-4070
>             Project: Traffic Server
>          Issue Type: Bug
>          Components: Network
>            Reporter: Craig Schomburg
>            Assignee: Eric Schwartz
>            Priority: Critical
>              Labels: regression
>             Fix For: 6.1.0
>
>         Attachments: TS-4070.patch
>
>
> During the rework of RemapProcessor.cc, RemapProcessor::setup_for_remap() as 
> part of the TS-2157 changeset, the port access API appears to have been 
> incorrectly modified to use the client_info.src_addr.host_order_port() API 
> [source port, host order] instead of the client_info.dst_addr.port() 
> [destination/receive port, network 
> order] API.  This caused port based remapping based on the receive port to 
> fail with ATS 6.0.0.
> Functionality was previously working with ATS 4.0.1.
> See attached patch that was used to restore the ATS 4.0.1 functionality.
> Looking for confirmation that this fix/patch was correct.



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

Reply via email to