You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2021/02/05 16:23:11 UTC

[GitHub] [trafficserver] rob05c commented on pull request #7483: Fix KA header not checking strategy

rob05c commented on pull request #7483:
URL: https://github.com/apache/trafficserver/pull/7483#issuecomment-774135659


   We had some discussions about `Proxy-Connection` in Slack. It is Always Wrong for ATS (or anyone) to _send_ Proxy-Connection. Though we should probably use it if we receive it.
   
   I made an issue to track that: https://github.com/apache/trafficserver/issues/7492
   
   But, this still fixes a bug in the logic which will lead to subtle and cryptic bugs, and fixes Strategies and non-Strategies to be consistent.
   
   I think we should merge this for the immediate bug, and then fix 7492 separately.


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