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 2020/11/30 17:15:50 UTC

[GitHub] [trafficserver] bneradt opened a new pull request #7359: Add negative caching tests and fixes.

bneradt opened a new pull request #7359:
URL: https://github.com/apache/trafficserver/pull/7359


   This adds test coverage for the negative caching feature and makes some
   fixes as a result of the test's findings.


----------------------------------------------------------------
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] bneradt commented on pull request #7359: Add negative caching tests and fixes.

Posted by GitBox <gi...@apache.org>.
bneradt commented on pull request #7359:
URL: https://github.com/apache/trafficserver/pull/7359#issuecomment-736033678


   > Am I missing the tests that the commit message talks about?
   
   Oh, good catch. That comment is a copy and paste from the master PR comment that contains the test:
   https://github.com/apache/trafficserver/commit/8eb68266167d8f8b3fa3a00ca9f6b7889e8ec101#diff-ee4f5dc00b390358f90ea4edbaa09f0130c95c646920fa25c9c0af9f72407533R1
   
   That test cannot be ported over into 8.1.x nor 7.1.x because it uses Proxy Verifier which is not supported in the AuTest extension in those branches.


----------------------------------------------------------------
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] bryancall removed a comment on pull request #7359: Add negative caching tests and fixes.

Posted by GitBox <gi...@apache.org>.
bryancall removed a comment on pull request #7359:
URL: https://github.com/apache/trafficserver/pull/7359#issuecomment-735952899


   [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] bryancall merged pull request #7359: Add negative caching tests and fixes.

Posted by GitBox <gi...@apache.org>.
bryancall merged pull request #7359:
URL: https://github.com/apache/trafficserver/pull/7359


   


----------------------------------------------------------------
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] bryancall commented on pull request #7359: Add negative caching tests and fixes.

Posted by GitBox <gi...@apache.org>.
bryancall commented on pull request #7359:
URL: https://github.com/apache/trafficserver/pull/7359#issuecomment-735952899


   [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