Uploaded image for project: 'Struts 2'
  1. Struts 2
  2. WW-3554

Tomcat requires JsonPlugin.tld to be in META-INF directory

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.2.1
    • Fix Version/s: 2.2.3
    • Component/s: Plugin - JSON
    • Labels:
      None

      Description

      JBoss doesn't seem to care. It finds the .tld even where it is now (top level). Tomcat, however, is picky. With the .tld in the top level directory, it doesn't find it. It works when I move the .tld into META-INF.

      To be clear:

      src/plugins/json/src/main/resources/JsonPlugin.tld -> src/plugins/json/src/main/resources/META-INF/JsonPlugin.tld

        Activity

        Hide
        newton_dave Dave Newton added a comment -

        Which server has the correct behavior?

        Show
        newton_dave Dave Newton added a comment - Which server has the correct behavior?
        Hide
        jafl5272 John Lindal added a comment -

        I believe Tomcat is correct. tld files should be in META-INF.

        But even if I'm wrong about that, if the package doesn't work with Tomcat, it's broken.

        Show
        jafl5272 John Lindal added a comment - I believe Tomcat is correct. tld files should be in META-INF. But even if I'm wrong about that, if the package doesn't work with Tomcat, it's broken.
        Hide
        newton_dave Dave Newton added a comment -

        In a loose sense, sure, but it's still good information to have, no?

        Show
        newton_dave Dave Newton added a comment - In a loose sense, sure, but it's still good information to have, no?
        Hide
        maurizio.cucchiara Maurizio Cucchiara added a comment -

        I realized this behaviour yesterday, Jetty works the same way as Tomcat and according to this documentation [1] this is the right behaviour.
        So, I'm going to apply John suggestion. Any thoughts on this?

        [1] http://download.oracle.com/javaee/5/tutorial/doc/bnamu.html#bnane

        Show
        maurizio.cucchiara Maurizio Cucchiara added a comment - I realized this behaviour yesterday, Jetty works the same way as Tomcat and according to this documentation [1] this is the right behaviour. So, I'm going to apply John suggestion. Any thoughts on this? [1] http://download.oracle.com/javaee/5/tutorial/doc/bnamu.html#bnane
        Hide
        pbenedict pbenedict added a comment -

        Yup, Tomcat follows the spec. JBoss must be a bit liberal for old applications.

        Show
        pbenedict pbenedict added a comment - Yup, Tomcat follows the spec. JBoss must be a bit liberal for old applications.

          People

          • Assignee:
            maurizio.cucchiara Maurizio Cucchiara
            Reporter:
            jafl5272 John Lindal
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development