Sort of unrelated to the patch but noticed from line:
File file = new File(configBase, name.replace('/', '_') + ".xml");Would the following 2 names cause a confict if used at the same time? 1) name="more_cowbell"; 2) name="more/cowbell";
-Tim (waiting for coffee to brew before trying to think)
[EMAIL PROTECTED] wrote:
remm 2003/07/14 03:54:11
Modified: catalina/src/share/org/apache/catalina/core
StandardContext.java
Log:
- Strip out leading '/' for the context file name, for cosmetic reasons.
Revision Changes Path
1.70 +4 -1 jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/core/StandardContext.java
Index: StandardContext.java
===================================================================
RCS file: /home/cvs/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/core/StandardContext.java,v
retrieving revision 1.69
retrieving revision 1.70
diff -u -r1.69 -r1.70
--- StandardContext.java 2 Jul 2003 22:00:34 -0000 1.69
+++ StandardContext.java 14 Jul 2003 10:54:11 -0000 1.70
@@ -3867,6 +3867,9 @@
if (name.equals("")) {
name = "ROOT";
}
+ if (name.startsWith("/")) {
+ name = name.substring(1);
+ }
File file = new File(configBase, name.replace('/', '_') + ".xml");
setConfigFile(file.getPath());
}
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
