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/27 00:15:19 UTC

[GitHub] [trafficserver] maskit commented on pull request #7882: Add an HTTP/2 304 "Not Modified" AuTest.

maskit commented on pull request #7882:
URL: https://github.com/apache/trafficserver/pull/7882#issuecomment-849204071


   Having more tests is a good thing so I'm not going to block this, however, I wouldn't put the test under h2 directory because 304 is not an H2 specific thing. I wrote this on some previous PR before, but I think we should put tests under h2 directory only if those check H2 specific behavior. Other tests should be under other directories accordingly and the tests should be ran on all the HTTP versions.


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