[MTOMCAT-269] Context file of additional web apps is ignored

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 2.2
• Component/s: tomcat6, tomcat7
• Assignee: Olivier Lamy
• Reporter: Stefan Kopf
• Created: 19/May/14 13:54
• Updated: 19/May/14 13:57

Description

While the default context file (/META-INF/context.xml) is respected for the project itself, it is ignored for the additional web apps.

org/apache/tomcat/maven/plugin/tomcat7/run/AbstractRunMojo.java line 1425

        for ( AbstractWebapp additionalWebapp : getAdditionalWebapps() )
        {
            String contextPath = additionalWebapp.getContextPath();
            if ( !contextPath.startsWith( "/" ) )
            {
                contextPath = "/" + contextPath;
            }
            addContextFromArtifact( container, contexts, getArtifact( additionalWebapp ), contextPath,
                                    additionalWebapp.getContextFile(), additionalWebapp.isAsWebapp() );
        }

Proposed fix:

org/apache/tomcat/maven/plugin/tomcat7/run/AbstractRunMojo.java Proposed fix

        for ( AbstractWebapp additionalWebapp : getAdditionalWebapps() )
        {
            String contextPath = additionalWebapp.getContextPath();
            if ( !contextPath.startsWith( "/" ) )
            {
                contextPath = "/" + contextPath;
            }
            String contextFile = null;
            if(additionalWebapp.getContextFile() != null)
            {
                contextFile = additionalWebapp.getContextFile();
            }
            else
            {
                File defaultConfigFile = new File( configurationDir, "webapps/"+artifact.getArtifactId()+"/META-INF/context.xml" );
                if(defaultConfigFile.exists())
                {
                    contextFile = defaultConfigFile.getAbsolutePath();
                }
            }
            addContextFromArtifact( container, contexts, getArtifact( additionalWebapp ), contextPath,
                                    contextFile, additionalWebapp.isAsWebapp() );
        }

These might be a similar problem with tomcat6-maven-plugin