[MEECROWAVE-86] Typo in TomcatAutoInitializer.java causes jsp pages not picked up by meecrowave.

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.2.0
• Fix Version/s: 1.2.0
• Assignee: Mark Struberg
• Reporter: Ravisankar Challa
• Created: 16/Dec/17 10:44
• Updated: 25/Dec/17 10:37
• Resolved: 16/Dec/17 15:49

Description

https://github.com/apache/meecrowave/blob/trunk/meecrowave-core/src/main/java/org/apache/meecrowave/tomcat/TomcatAutoInitializer.java
Typo in TomcatAutoInitializer.java causes jsp pages processed by Tomcat Default Servlet instead of Tomcat Jsp Servlet.

final ServletRegistration.Dynamic def = ctx.addServlet("default", DefaultServlet.class);
def.setInitParameter("listings", "false");
def.setInitParameter("debug", "0");
def.setLoadOnStartup(1);
def.addMapping("/");

try {
    final String jsp = "org.apache.jasper.servlet.JspServlet";
    TomcatAutoInitializer.class.getClassLoader().loadClass(jsp);
    final ServletRegistration.Dynamic jspDef = ctx.addServlet("jsp", jsp);
    if (jspDef != null) {
        jspDef.setInitParameter("fork", "false");
        jspDef.setInitParameter("xpoweredBy", "false");
        jspDef.setInitParameter("development", "false");
        jspDef.setLoadOnStartup(3);
        def.addMapping("*.jsp");  // Should be --- jspDef.addMapping("*.jsp");
        def.addMapping("*.jspx"); // Should be --- jspDef.addMapping("*.jspx");
    }