You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2013/11/25 20:43:06 UTC

svn commit: r1545377 - in /tomcat/trunk: test/org/apache/catalina/startup/TestHostConfigAutomaticDeployment.java webapps/docs/config/automatic-deployment.xml webapps/docs/config/host.xml

Author: markt
Date: Mon Nov 25 19:43:06 2013
New Revision: 1545377

URL: http://svn.apache.org/r1545377
Log:
Update unit tests and documentation now that deployment fails if deployXML is true and the application has an embedded context.xml

Modified:
    tomcat/trunk/test/org/apache/catalina/startup/TestHostConfigAutomaticDeployment.java
    tomcat/trunk/webapps/docs/config/automatic-deployment.xml
    tomcat/trunk/webapps/docs/config/host.xml

Modified: tomcat/trunk/test/org/apache/catalina/startup/TestHostConfigAutomaticDeployment.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/startup/TestHostConfigAutomaticDeployment.java?rev=1545377&r1=1545376&r2=1545377&view=diff
==============================================================================
--- tomcat/trunk/test/org/apache/catalina/startup/TestHostConfigAutomaticDeployment.java (original)
+++ tomcat/trunk/test/org/apache/catalina/startup/TestHostConfigAutomaticDeployment.java Mon Nov 25 19:43:06 2013
@@ -335,28 +335,28 @@ public class TestHostConfigAutomaticDepl
     public void testDeploymentWarXmlFFF() throws Exception {
         createWar(WAR_XML_SOURCE, true);
         doTestDeployment(false, false, false,
-                LifecycleState.STARTED, null, false, true, false);
+                LifecycleState.FAILED, null, false, true, false);
     }
 
     @Test
     public void testDeploymentWarXmlFFT() throws Exception {
         createWar(WAR_XML_SOURCE, true);
         doTestDeployment(false, false, true,
-                LifecycleState.STARTED, null, false, true, true);
+                LifecycleState.FAILED, null, false, true, true);
     }
 
     @Test
     public void testDeploymentWarXmlFTF() throws Exception {
         createWar(WAR_XML_SOURCE, true);
         doTestDeployment(false, true, false,
-                LifecycleState.STARTED, null, false, true, false);
+                LifecycleState.FAILED, null, false, true, false);
     }
 
     @Test
     public void testDeploymentWarXmlFTT() throws Exception {
         createWar(WAR_XML_SOURCE, true);
         doTestDeployment(false, true, true,
-                LifecycleState.STARTED, null, false, true, true);
+                LifecycleState.FAILED, null, false, true, true);
     }
 
     @Test
@@ -462,28 +462,28 @@ public class TestHostConfigAutomaticDepl
     public void testDeploymentDirXmlFFF() throws Exception {
         createDirInAppbase(true);
         doTestDeployment(false, false, false,
-                LifecycleState.STARTED, null, false, false, true);
+                LifecycleState.FAILED, null, false, false, true);
     }
 
     @Test
     public void testDeploymentDirXmlFFT() throws Exception {
         createDirInAppbase(true);
         doTestDeployment(false, false, true,
-                LifecycleState.STARTED, null, false, false, true);
+                LifecycleState.FAILED, null, false, false, true);
     }
 
     @Test
     public void testDeploymentDirXmlFTF() throws Exception {
         createDirInAppbase(true);
         doTestDeployment(false, true, false,
-                LifecycleState.STARTED, null, false, false, true);
+                LifecycleState.FAILED, null, false, false, true);
     }
 
     @Test
     public void testDeploymentDirXmlFTT() throws Exception {
         createDirInAppbase(true);
         doTestDeployment(false, true, true,
-                LifecycleState.STARTED, null, false, false, true);
+                LifecycleState.FAILED, null, false, false, true);
     }
 
     @Test

Modified: tomcat/trunk/webapps/docs/config/automatic-deployment.xml
URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/automatic-deployment.xml?rev=1545377&r1=1545376&r2=1545377&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/config/automatic-deployment.xml (original)
+++ tomcat/trunk/webapps/docs/config/automatic-deployment.xml Mon Nov 25 19:43:06 2013
@@ -527,8 +527,9 @@
     <li><em>unpackWARs</em> is ignored since there is no WAR file.</li>
     <li>The context will fail to start because there is no content in the
         expected <em>docBase</em>.</li>
-    <li>The embedded META-INF/context.xml is ignored because <em>deployXML</em>
-        is <code>false</code>.</li>
+    <li>The web application fails to deploy because it contains an embedded
+        META-INF/context.xml, <em>deployXML</em> is <code>false</code> and an
+        XML has not been provided in the <em>configBase</em>.</li>
     <li>The XML file is only deleted if <em>copyXML</em> is <code>true</code>
         and <em>deployXML</em> is <code>true</code>.</li>
     <li>Although the external resource is still present, the web application is

Modified: tomcat/trunk/webapps/docs/config/host.xml
URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/host.xml?rev=1545377&r1=1545376&r2=1545377&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/config/host.xml (original)
+++ tomcat/trunk/webapps/docs/config/host.xml Mon Nov 25 19:43:06 2013
@@ -241,9 +241,13 @@
         interacting with the container's configuration. The  administrator will
         then be responsible for providing an external context configuration
         file, and putting it in the location defined by the
-        <strong>xmlBase</strong> attribute. The flag's value defaults to
-        <code>true</code> unless a security manager is enabled when the default
-        is <code>false</code>.</p>
+        <strong>xmlBase</strong> attribute. If this flag is <code>false</code>,
+        a descriptor is located at <code>/META-INF/context.xml</code> and no
+        descriptor is present in <strong>xmlBase</strong> then the context will
+        fail to start in case the descriptor contains necessary configuration
+        for secure deployment (such as a RemoteAddrValve) which should not be
+        ignored. The flag's value defaults to <code>true</code> unless a
+        security manager is enabled when the default is <code>false</code>.</p>
       </attribute>
 
       <attribute name="errorReportValveClass" required="false">



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org