arturobernalg opened a new pull request, #1163: URL: https://github.com/apache/tomcat/pull/1163
RFC 9110 section 15.4.5 requires a 304 response to include the Vary header field that would have been sent in a 200 response to the same request. When precompressed resources are available, DefaultServlet adds Vary: accept-encoding to the response, but only after it has checked the If headers. A 304 response for If-None-Match or If-Modified-Since was therefore sent without it, although the 200 response has it. The header is now added before the If headers are checked. The list of precompressed resources is only looked up once. The new test fails without the change and passes with it. TestDefaultServlet, TestDefaultServletRfc9110Section13, TestDefaultServletIfMatchRequests and TestDefaultServletRangeRequests pass. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
