[GitHub] [trafficserver] bneradt opened a new issue, #9335: Proxy Protocol strings have dest and src reversed

["bneradt (via GitHub)" <gi...@apache.org>]

bneradt opened a new issue, #9335:
URL: https://github.com/apache/trafficserver/issues/9335

   I noticed that when we configure ATS to send Proxy Protocol via [proxy.config.http.proxy_protocol_out](https://docs.trafficserver.apache.org/admin-guide/files/records.config.en.html#proxy-config-http-proxy-protocol-out), our Proxy Protocol strings have src and dest descriptions reversed:
   
   https://github.com/apache/trafficserver/blob/3dcf2a8a0fae7d1ca2140af60b55ea50b8d2b4ec/proxy/http/HttpSM.cc#L132
   
   We should pass `vc_in->get_remote_endpoint()` (which is the client's src address) before `local` (which is the client's dest address).
   
   I have an internal Yahoo patch I'm working on for this with an AuTest that I plan to opensource soon.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficserver] bneradt closed issue #9335: Proxy Protocol strings have dest and src reversed

["bneradt (via GitHub)" <gi...@apache.org>]

bneradt closed issue #9335: Proxy Protocol strings have dest and src reversed
URL: https://github.com/apache/trafficserver/issues/9335


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org