zwoop commented on PR #12373:
URL: https://github.com/apache/trafficserver/pull/12373#issuecomment-3102046878

   Two thoughts:
   
   1. This doesn't seem complete, assuming it's right, since these parameters 
are used in several places of the HttpSM.
   2. This was change ~4 years ago, and as far as I can tell, the PR here 
reverses the changes made then.
   
   ```
   commit 1c3a1a41bbf90581a137505f55e06d98fe6aabe1
   Author:     Damian Meden <damian.me...@gmail.com>
   AuthorDate: Thu May 27 21:06:39 2021 +0100
   Commit:     GitHub <nore...@github.com>
   CommitDate: Thu May 27 13:06:39 2021 -0700
   
       Redirect - Make TS to honour the number_of_redirections configuration 
value (#7867)
       
       This fix includes a unit test to validate the expected behavior.
   ```
   
   We'll have to have Damien look at this I think.


-- 
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-unsubscr...@trafficserver.apache.org

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

Reply via email to