stokito opened a new pull request #324:
URL: https://github.com/apache/tomcat/pull/324


   Currently Tomcat 9 generates ETag like `W/"1047-1578315296666"` i.e. 
`Weak"Size-MTime in Milliseconds"`.
   This is incorrect ETag because it should be strong as for a static file i.e. 
octal compatibility.
   Also to make ETag working for load balancing between Tomcat and Nginx we 
need to change format to `"hex(MTime in seconds)-hex(Size)"` e.g. 
`"5e132e20-417"`.
   Nginx is not configurable but ether Tomcat is not. In the same time Nginx is 
more widely used while Tomcat returns incorrect ETag.
   This small PR fixes the problem.
   As a downside after update Tomcat will discard existing ETags but then 
they'll be updated on clients and everything will work fine.
   


----------------------------------------------------------------
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to