Chenjp commented on PR #807:
URL: https://github.com/apache/tomcat/pull/807#issuecomment-2592561309
> I see a few things to cleanup as well.
+1.
rename method is better.
about testcase: pls try to cover allowSpecialPaths and nonroot webdav path.
Current testcase is not enoug
rmaucher commented on PR #807:
URL: https://github.com/apache/tomcat/pull/807#issuecomment-2592187806
I did not pick up the test since it doesn't add much.
Also because of:
File appDir = new File(getBuildDirectory(), "webapps" + contextPath);
try (FileWriter fw = new
rmaucher closed pull request #807: Webdav special path handling fix
URL: https://github.com/apache/tomcat/pull/807
--
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 unsubscrib
rmaucher commented on PR #807:
URL: https://github.com/apache/tomcat/pull/807#issuecomment-2592078702
Ok, the check should be similar to what it is in StandardContextValve. I
won't rename the method however, and I see a few things to cleanup as well.
--
This is an automated message from t
Chenjp opened a new pull request, #807:
URL: https://github.com/apache/tomcat/pull/807
1. fix: "/web-information.xml" is not a special path.
2. method name refactoring: from isSpecialPath to checkSpecialPath.
--
This is an automated message from the Apache Git Service.
To respond to th
