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/05/21 15:04:06 UTC
[GitHub] [trafficserver] brbzull0 opened a new pull request #7867: Redirect - Make TS to honour the number_of_redirections configuration.
brbzull0 opened a new pull request #7867:
URL: https://github.com/apache/trafficserver/pull/7867
This fix includes a unit test to validate the expected behavior.
This fixes https://github.com/apache/trafficserver/issues/7800 and will help to validate https://github.com/apache/trafficserver/issues/2742
I would appreciate if you can review it carefully, I did test this but I may be missing some scenarios, I'll be happy to add them to another unit test if necesary.
This PR depends on the following https://github.com/apache/trafficserver/pull/7866 to work. I will rebase once https://github.com/apache/trafficserver/pull/7866 is merged back to master
--
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
[GitHub] [trafficserver] brbzull0 commented on pull request #7867: Redirect - Make TS to honour the number_of_redirections configuration.
Posted by GitBox <gi...@apache.org>.
brbzull0 commented on pull request #7867:
URL: https://github.com/apache/trafficserver/pull/7867#issuecomment-847237046
[approve ci autest]
--
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
[GitHub] [trafficserver] brbzull0 removed a comment on pull request #7867: Redirect - Make TS to honour the number_of_redirections configuration.
Posted by GitBox <gi...@apache.org>.
brbzull0 removed a comment on pull request #7867:
URL: https://github.com/apache/trafficserver/pull/7867#issuecomment-847236700
[appove ci autest]
--
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
[GitHub] [trafficserver] brbzull0 removed a comment on pull request #7867: Redirect - Make TS to honour the number_of_redirections configuration.
Posted by GitBox <gi...@apache.org>.
brbzull0 removed a comment on pull request #7867:
URL: https://github.com/apache/trafficserver/pull/7867#issuecomment-847237046
[approve ci autest]
--
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
[GitHub] [trafficserver] brbzull0 commented on pull request #7867: Redirect - Make TS to honour the number_of_redirections configuration.
Posted by GitBox <gi...@apache.org>.
brbzull0 commented on pull request #7867:
URL: https://github.com/apache/trafficserver/pull/7867#issuecomment-847257867
[approve ci autest]
--
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
[GitHub] [trafficserver] zwoop commented on pull request #7867: Redirect - Make TS to honour the number_of_redirections configuration.
Posted by GitBox <gi...@apache.org>.
zwoop commented on pull request #7867:
URL: https://github.com/apache/trafficserver/pull/7867#issuecomment-893592447
Cherry-picked to v9.1.x branch.
--
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
[GitHub] [trafficserver] randall merged pull request #7867: Redirect - Make TS to honour the number_of_redirections configuration.
Posted by GitBox <gi...@apache.org>.
randall merged pull request #7867:
URL: https://github.com/apache/trafficserver/pull/7867
--
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
[GitHub] [trafficserver] brbzull0 edited a comment on pull request #7867: Redirect - Make TS to honour the number_of_redirections configuration.
Posted by GitBox <gi...@apache.org>.
brbzull0 edited a comment on pull request #7867:
URL: https://github.com/apache/trafficserver/pull/7867#issuecomment-846053625
~~Autest needs https://github.com/apache/trafficserver/pull/7866 to work, as includes a new feature which is vital for this PR's unit test to work.~~ Done
--
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
[GitHub] [trafficserver] brbzull0 commented on pull request #7867: Redirect - Make TS to honour the number_of_redirections configuration.
Posted by GitBox <gi...@apache.org>.
brbzull0 commented on pull request #7867:
URL: https://github.com/apache/trafficserver/pull/7867#issuecomment-846053625
Autest needs https://github.com/apache/trafficserver/pull/7866 to work, as includes a new feature which is vital for this PR's unit test to work.
--
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
[GitHub] [trafficserver] brbzull0 commented on pull request #7867: Redirect - Make TS to honour the number_of_redirections configuration.
Posted by GitBox <gi...@apache.org>.
brbzull0 commented on pull request #7867:
URL: https://github.com/apache/trafficserver/pull/7867#issuecomment-847236700
[appove ci autest]
--
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