Mule
  1. Mule
  2. MULE-5100

Verbatim tomcat dependencies must not be used

    Details

    • User impact:
      Medium
    • Similar Issues:
      MULE-7789Update mule-transports-http to tomcat 6+
      MULE-7684Enable dependency injection on registry elements
      MULE-5229Tomcat service deployment documentation is not complete
      MULE-8154Module: Tomcat hard-coded version - use defined version
      MULE-2822Simplify FixedHostRmiClientSocketFactory to have no extra dependencies
      MULE-2128SwitchVersion.groovy must also update version numbers of dependencies on mule artifacts
      MULE-4776webapp example fails when using tomcat:run
      MULE-2700Mule must not add xml-apis/xerces to the classpath
      MULE-6705NoSuchMethodError in Tomcat embedded MULE when executing http:set-cookie
      MULE-4325Tomcat converts all Header-Names to lowercase

      Description

      Recent cookie enhancements in http transport added 2 tomcat depencies (tomcat-util & tomcat-apr), totalling about 300KB extra.

      1. WHY tomcat 5.5, and not 6.0?
      2. Without jar shading, it WILL conflict with tomcat jars bundled in jboss. What version wins and what hidden regressions may come out of it?

      This should be reworked to use the latest available stable tomcat AND actual classes identified and bytecode-shaded (so they'll be in org.mule.org.apache.tomcat.util for example). This will guarantee to library version conflicts. Take a look at what e.g. MMC agent build does with some classes that we pull from the ant.jar.

        Issue Links

          Activity

          Hide
          Dirk Olmes added a comment -

          IIRC Edu introduced the tomcat dependencies.

          Show
          Dirk Olmes added a comment - IIRC Edu introduced the tomcat dependencies.
          Hide
          Jeremy Landis added a comment -

          Please close this jira. It was resolved in MULE-7789.

          Show
          Jeremy Landis added a comment - Please close this jira. It was resolved in MULE-7789 .

            People

            • Assignee:
              Edu Pereda
              Reporter:
              Andrew Perepelytsya
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:

                Development