markt-asf commented on PR #816:
URL: https://github.com/apache/tomcat/pull/816#issuecomment-2640639410

   I think I understand everything that is going on now.
   
   The root cause is that Tomcat accepts name for 
`ClassLoader.getResource(String name)` and friends with a leading '/' whereas 
the JRE provided class loaders do not.
   
   My plan to deal with this is:
   - Tomcat 12 onwards return `null` if the name starts with `/`
   - Tomcat 11 and earlier, skip the `notFoundClassResources` cache if the name 
starts with '/'
   
   Apps that use incorrect resource names may see a small performance 
degradation if they try loading the same resource multiple times using the 
wrong format for a resource name. The fix for that would be for the app to use 
the correct format.


-- 
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: dev-unsubscr...@tomcat.apache.org

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