[GitHub] [trafficserver] jp557198 commented on pull request #7069: Move the direct self loop check later to HttpSM::do_http_server_open

[GitBox <gi...@apache.org>]

jp557198 commented on pull request #7069:
URL: https://github.com/apache/trafficserver/pull/7069#issuecomment-668258578


   ATS 7.1.4
   
   As a fyi..  we ran into this problem during our edge peering lab testing. This was my observation:
   
   "Proxy cycle is occurring because the proxy is making a request to itself.. ATS code compares the server and client via string values.
   See the below traffic.out and corresponding code.
   Maybe we can try playing with the via string(proxy_request_via_string)"
   
   To 'fix' this issue, we simply set proxy_request_via_string to 0.  Once set, a child was able to peer to itself.
   
   Side note: Edge/Child peering is currently soaking in prod and has worked to significantly increase HIT rate, as well as lower storage requirements(content/library sharding)
   
   
   


----------------------------------------------------------------
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.

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