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