eduarddrenth commented on code in PR #19: URL: https://github.com/apache/velocity-tools/pull/19#discussion_r1683792113
########## velocity-tools-view/src/main/java/org/apache/velocity/tools/view/ServletUtils.java: ########## @@ -80,9 +80,9 @@ public static String getPath(HttpServletRequest request) // will return the original (wrong) URI requested. The following special // attribute holds the correct path. See section 8.3 of the Servlet // 2.3 specification. - String path = (String)request.getAttribute("javax.servlet.include.servlet_path"); + String path = (String)request.getAttribute("jakarta.servlet.include.servlet_path"); // also take into account the PathInfo stated on SRV.4.4 Request Path Elements Review Comment: done. I also noticed that a perhaps bigger issue in the future will be to refactor the security setup -- 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...@velocity.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@velocity.apache.org For additional commands, e-mail: dev-h...@velocity.apache.org