[jira] [Created] (MTOMCAT-270) Additional web apps are not re-deployed properly

["Stefan Kopf (JIRA)" <ji...@apache.org>]

Stefan Kopf created MTOMCAT-270:
-----------------------------------

             Summary: Additional web apps are not re-deployed properly
                 Key: MTOMCAT-270
                 URL: https://issues.apache.org/jira/browse/MTOMCAT-270
             Project: Apache Tomcat Maven Plugin
          Issue Type: Bug
          Components: tomcat6, tomcat7
    Affects Versions: 2.2
            Reporter: Stefan Kopf
            Assignee: Olivier Lamy (*$^¨%`£)


Additional Web apps are exploded into a directory under the webapps folder in the Tomcat config directory - but only if this folder does not yet exist.

{code:org/apache/tomcat/maven/plugin/tomcat7/run/AbstractRunMojo.java line 1443|borderStyle=solid}
        File webapps = new File( configurationDir, "webapps" );
        File artifactWarDir = new File( webapps, artifact.getArtifactId() );
        if ( !artifactWarDir.exists() )
        {
            //dont extract if exists
            artifactWarDir.mkdir();
            ...
{code}

If the artifact is changed (i.e. when deploying a snapshot), it is not redeployed since the directory does already exist.

Since the directory is just named after the artifact id and does not contain group id or version, it is also not re deployed even if the version of the dependency is changed.

Proposed fix (simple):
If the directory exists, it should be removed before exploding the web app.
Problem:
This does work correctly, but will cause some overhead since it will be always re-deploying.

Proposed optimal fix:
1. The deployment directory where the webapp is exploded to should consist of groupId, artifactId and version, and not just the artifact id.
For example:
{code:org/apache/tomcat/maven/plugin/tomcat7/run/AbstractRunMojo.java suggested change|borderStyle=solid}
     private void addContextFromArtifact( Tomcat container, List<Context> contexts, Artifact artifact,
                                         String contextPath, File contextXml, boolean asWebApp )
        throws MojoExecutionException, ServletException, IOException
    {
        getLog().info( "Deploy warfile: " + String.valueOf( artifact.getFile() ) + " to contextPath: " + contextPath );
        File webapps = new File( configurationDir, "webapps" );
        File artifactWarDir = new File( webapps, artifact.getGroupId() + "/" + artifact.getArtifactId() + "/" + artifact.getVersion() );
        if ( !artifactWarDir.exists() )
        ...
{code}
2. The timestamp of the directory should be compared against the timestamp of the war file. If the war file is newer, the directory is deleted and redeployed:
{code:org/apache/tomcat/maven/plugin/tomcat7/run/AbstractRunMojo.java suggested change|borderStyle=solid}
        getLog().info( "Deploy warfile: " + String.valueOf( artifact.getFile() ) + " to contextPath: " + contextPath );
        File webapps = new File( configurationDir, "webapps" );
        File artifactWarDir = new File( webapps, artifact.getGroupId() + "/" + artifact.getArtifactId() + "/" + artifact.getVersion() );
        if( artifactWarDir.exists() )
        {
            if( artifact.getFile().lastModified() > artifactWarDir.lastModified() )
            {
                FileUtils.deleteDirectory( artifactWarDir );
            }
        }
        if ( !artifactWarDir.exists() )
        {
            //dont extract if exists
            artifactWarDir.mkdir();
            try
            {
{code}

Please note that the changed directory layout proposed by this fix influences the fix proposed in MTOMCAT-269



--
This message was sent by Atlassian JIRA
(v6.2#6252)

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