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 2015/03/04 23:35:12 UTC
svn commit: r1664172 - in /tomcat/trunk:
java/org/apache/catalina/startup/HostConfig.java
webapps/docs/config/valve.xml
Author: markt
Date: Wed Mar 4 22:35:11 2015
New Revision: 1664172
URL: http://svn.apache.org/r1664172
Log:
Follow-up to r1663995.
kkolino review
Add another link. Fix a typo.
Modified:
tomcat/trunk/java/org/apache/catalina/startup/HostConfig.java
tomcat/trunk/webapps/docs/config/valve.xml
Modified: tomcat/trunk/java/org/apache/catalina/startup/HostConfig.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/startup/HostConfig.java?rev=1664172&r1=1664171&r2=1664172&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/startup/HostConfig.java (original)
+++ tomcat/trunk/java/org/apache/catalina/startup/HostConfig.java Wed Mar 4 22:35:11 2015
@@ -598,13 +598,14 @@ public class HostConfig
}
}
+ boolean unpackWAR = unpackWARs;
+ if (unpackWAR && context instanceof StandardContext) {
+ unpackWAR = ((StandardContext) context).getUnpackWAR();
+ }
+
// Add the eventual unpacked WAR and all the resources which will be
// watched inside it
if (isExternalWar) {
- boolean unpackWAR = unpackWARs;
- if (unpackWAR && context instanceof StandardContext) {
- unpackWAR = ((StandardContext) context).getUnpackWAR();
- }
if (unpackWAR) {
deployedApp.redeployResources.put(expandedDocBase.getAbsolutePath(),
Long.valueOf(expandedDocBase.lastModified()));
@@ -626,7 +627,7 @@ public class HostConfig
Long.valueOf(0));
}
}
- if (expandedDocBase.exists()) {
+ if (unpackWAR) {
deployedApp.redeployResources.put(expandedDocBase.getAbsolutePath(),
Long.valueOf(expandedDocBase.lastModified()));
addWatchedResources(deployedApp,
@@ -634,8 +635,9 @@ public class HostConfig
} else {
addWatchedResources(deployedApp, null, context);
}
- // Add the context XML to the list of files which should trigger a redeployment
if (!isExternal) {
+ // For external docBases, the context.xml will have been
+ // added above.
deployedApp.redeployResources.put(
contextXml.getAbsolutePath(),
Long.valueOf(contextXml.lastModified()));
Modified: tomcat/trunk/webapps/docs/config/valve.xml
URL: http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/config/valve.xml?rev=1664172&r1=1664171&r2=1664172&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/config/valve.xml (original)
+++ tomcat/trunk/webapps/docs/config/valve.xml Wed Mar 4 22:35:11 2015
@@ -343,10 +343,11 @@
logged by the Access Log Valve may represent the reverse proxy, the browser
or some combination of the two depending on the configuration of Tomcat and
the reverse proxy. For Tomcat configuration options see
- <a href="#Proxies_Support">Proxies Support</a>. For reverse proxies that
+ <a href="#Proxies_Support">Proxies Support</a> and the
+ <a href="../proxy-howto.html">Proxy How-To</a>. For reverse proxies that
use mod_jk, see the <a
href="http://tomcat.apache.org/connectors-doc/generic_howto/proxy.html">generic
- proxy</a> docmentation. For other reverse proxies, consult their
+ proxy</a> documentation. For other reverse proxies, consult their
documentation.</p>
</subsection>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org