tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject svn commit: r1345359 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/catalina/core/ webapps/docs/changelog.xml
Date Fri, 01 Jun 2012 21:42:24 GMT
Author: kkolinko
Date: Fri Jun  1 21:42:24 2012
New Revision: 1345359

Merged revision 1345357 from tomcat/trunk:
Review of r1344902:
restore passing DirContextURLStreamHandler instance into the new URL.

Note that we use "new DirContextURLStreamHandler(DirContext)" and
thus the handler will not need to call DirContextURLStreamHandler#get() to retrieve the context.

    tomcat/tc7.0.x/trunk/   (props changed)

Propchange: tomcat/tc7.0.x/trunk/
  Merged /tomcat/trunk:r1345357

Modified: tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/
--- tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/ (original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/catalina/core/ Fri Jun  1
21:42:24 2012
@@ -67,6 +67,7 @@ import org.apache.catalina.connector.Con
 import org.apache.catalina.deploy.FilterDef;
 import org.apache.catalina.util.ResourceSet;
 import org.apache.catalina.util.ServerInfo;
+import org.apache.naming.resources.DirContextURLStreamHandler;
 import org.apache.naming.resources.Resource;
 import org.apache.tomcat.util.ExceptionUtils;
 import org.apache.tomcat.util.buf.CharChunk;
@@ -533,8 +534,9 @@ public class ApplicationContext
             String hostName = context.getParent().getName();
             try {
-                return new URI("jndi",
-                        getJNDIUri(hostName, fullPath), null).toURL();
+                URI uri = new URI("jndi", getJNDIUri(hostName, fullPath), null);
+                return new URL(null, uri.toString(),
+                        new DirContextURLStreamHandler(resources));
             } catch (NamingException e) {
                 // Ignore
             } catch (Exception e) {

Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Fri Jun  1 21:42:24 2012
@@ -146,14 +146,15 @@
         <code>JreMemoryLeakPreventionListener</code>. (markt)
-        <bug>53257</bug>: Ensure that resources, inclusing JSP files, that have
+        <bug>53257</bug>: Ensure that resources, including JSP files, that have
         names that include characters with special meanings in URLs (such as
         ampersand, semicolon, plus, hash and percent) are correctly handled.
         This bug is partially a regression caused by the original fix for
         <bug>51584</bug> and partially an existing issue that had not previously
         been identified. This fix reverts the original fix for <bug>51584</bug>,
         correctly fixes that issue and fixes the additional issues identified by
-        the test cases that were also added as part of this fix. (markt)
+        the test cases that were also added as part of this fix.
+        (markt/kkolinko)
         <bug>53267</bug>: Ensure that using the GC Daemon Protection feature

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message