You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by kf...@apache.org on 2015/03/03 05:24:18 UTC
svn commit: r1663524 - in /tomcat/tc7.0.x/trunk:
java/org/apache/catalina/startup/HostConfig.java webapps/docs/changelog.xml
Author: kfujino
Date: Tue Mar 3 04:24:18 2015
New Revision: 1663524
URL: http://svn.apache.org/r1663524
Log:
Make sure that unpackWAR attribute of Context is handled correctly in HostConfig.
Modified:
tomcat/tc7.0.x/trunk/java/org/apache/catalina/startup/HostConfig.java
tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
Modified: tomcat/tc7.0.x/trunk/java/org/apache/catalina/startup/HostConfig.java
URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/java/org/apache/catalina/startup/HostConfig.java?rev=1663524&r1=1663523&r2=1663524&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/java/org/apache/catalina/startup/HostConfig.java (original)
+++ tomcat/tc7.0.x/trunk/java/org/apache/catalina/startup/HostConfig.java Tue Mar 3 04:24:18 2015
@@ -699,7 +699,11 @@ public class HostConfig
// Add the eventual unpacked WAR and all the resources which will be
// watched inside it
if (isExternalWar) {
- if (unpackWARs) {
+ boolean unpackWAR = unpackWARs;
+ if (unpackWAR && context instanceof StandardContext) {
+ unpackWAR = ((StandardContext) context).getUnpackWAR();
+ }
+ if (unpackWAR) {
deployedApp.redeployResources.put(expandedDocBase.getAbsolutePath(),
Long.valueOf(expandedDocBase.lastModified()));
addWatchedResources(deployedApp, expandedDocBase.getAbsolutePath(), context);
@@ -779,7 +783,11 @@ public class HostConfig
}
if (deploymentExists(cn.getName())) {
DeployedApplication app = deployed.get(cn.getName());
- if (!unpackWARs && app != null) {
+ boolean unpackWAR = unpackWARs;
+ if (unpackWAR && host.findChild(cn.getName()) instanceof StandardContext) {
+ unpackWAR = ((StandardContext) host.findChild(cn.getName())).getUnpackWAR();
+ }
+ if (!unpackWAR && app != null) {
// Need to check for a directory that should not be
// there
File dir = new File(appBase, cn.getBaseName());
@@ -1085,7 +1093,11 @@ public class HostConfig
} finally {
// If we're unpacking WARs, the docBase will be mutated after
// starting the context
- if (unpackWARs && context != null && context.getDocBase() != null) {
+ boolean unpackWAR = unpackWARs;
+ if (unpackWAR && context instanceof StandardContext) {
+ unpackWAR = ((StandardContext) context).getUnpackWAR();
+ }
+ if (unpackWAR && context != null && context.getDocBase() != null) {
File docBase = new File(appBase(), cn.getBaseName());
deployedApp.redeployResources.put(docBase.getAbsolutePath(),
Long.valueOf(docBase.lastModified()));
@@ -1412,7 +1424,11 @@ public class HostConfig
app.redeployResources.put(resources[i],
Long.valueOf(resource.lastModified()));
app.timestamp = System.currentTimeMillis();
- if (unpackWARs) {
+ boolean unpackWAR = unpackWARs;
+ if (unpackWAR && context instanceof StandardContext) {
+ unpackWAR = ((StandardContext) context).getUnpackWAR();
+ }
+ if (unpackWAR) {
addWatchedResources(app, context.getDocBase(), context);
} else {
addWatchedResources(app, null, context);
@@ -1759,7 +1775,11 @@ public class HostConfig
host.addChild(context);
// Add the eventual unpacked WAR and all the resources which will be
// watched inside it
- if (isWar && unpackWARs) {
+ boolean unpackWAR = unpackWARs;
+ if (unpackWAR && context instanceof StandardContext) {
+ unpackWAR = ((StandardContext) context).getUnpackWAR();
+ }
+ if (isWar && unpackWAR) {
File docBase = new File(appBase(), context.getBaseName());
deployedApp.redeployResources.put(docBase.getAbsolutePath(),
Long.valueOf(docBase.lastModified()));
Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml
URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml?rev=1663524&r1=1663523&r2=1663524&view=diff
==============================================================================
--- tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Tue Mar 3 04:24:18 2015
@@ -126,6 +126,10 @@
<bug>57637</bug>: Do not create unnecessary sessions when using
PersistentValve. (jboynes/fschumacher)
</fix>
+ <fix>
+ Make sure that <code>unpackWAR</code> attribute of <code>Context</code>
+ is handled correctly in <code>HostConfig</code>. (kfujino)
+ </fix>
</changelog>
</subsection>
<subsection name="Coyote">
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org