Jetspeed 2
  1. Jetspeed 2
  2. JS2-281

*.xml in jetspeed-security-*.jar cause problems

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-M3
    • Fix Version/s: 2.0-M4
    • Component/s: Security
    • Labels:
      None
    • Environment:
      JBoss

      Description

      The component file jetspeed-security-*.jar contains in its META-INF/ directory two *.xml files. These files prevent the deployment of this Jar in JBoss in other contexts than a WAR.

      The reason is that JBoss assumes that there is a special deployer for Jars that have .xml files in META-INF. So the default deployer for Jars skips any such Jars. This problem does not show if jakarta-security-.jar is put in the WEB-INF/lib simply because you cannot use the JBoss deployer for Jetspeed2 anyway, you must use the Tomcat deployer. but if you try to put jetspeed-security-*.jar in e.g. a sar, the problem shows.

      As far as I have understood, the *.xml files are only significant for testing anyway. They should be put elsewhere.

      I'm aware that this is, at its root, a JBoss problem. But I won't be able to make them change this behaviour.

        Activity

        Hide
        David Le Strat added a comment -

        Change committed. This should finally resolve this issue.

        Show
        David Le Strat added a comment - Change committed. This should finally resolve this issue.
        Hide
        David Le Strat added a comment -

        Will use JETSPEED-INF instead as requested by Scott.

        Show
        David Le Strat added a comment - Will use JETSPEED-INF instead as requested by Scott.
        Hide
        David Le Strat added a comment -

        The previous fix did not completely fix the issue. I propose to put Jetspeed configuration file into J2-INF instead of META-INF. This will avoid any issue of that sort. Please let me know what you think.

        Show
        David Le Strat added a comment - The previous fix did not completely fix the issue. I propose to put Jetspeed configuration file into J2-INF instead of META-INF. This will avoid any issue of that sort. Please let me know what you think.
        Hide
        Michael Lipp added a comment -

        Regrettably, it does not fix the issue as JBoss searches for *.xml files in META-INF recursively!

        I think the proper view of things is that META-INF should be considered a directory under control of the Java/J2EE specs and should therefore not be used by any other frameworks (though it has this tempting generic name). Something like OBJ-INF or SPRING-INF should be used instead.

        Show
        Michael Lipp added a comment - Regrettably, it does not fix the issue as JBoss searches for *.xml files in META-INF recursively! I think the proper view of things is that META-INF should be considered a directory under control of the Java/J2EE specs and should therefore not be used by any other frameworks (though it has this tempting generic name). Something like OBJ-INF or SPRING-INF should be used instead.
        Hide
        David Le Strat added a comment -

        All *.xml have been move from META-INF to META-INF/xxx/ for all components. This should fix this issue.

        Show
        David Le Strat added a comment - All *.xml have been move from META-INF to META-INF/xxx/ for all components. This should fix this issue.
        Hide
        David Le Strat added a comment -

        I started to roll this change out. OJB mapping will go under META-INF/ojb/ and OJB mapping files will have an _repository suffix.

        Show
        David Le Strat added a comment - I started to roll this change out. OJB mapping will go under META-INF/ojb/ and OJB mapping files will have an _repository suffix.
        Hide
        Michael Lipp added a comment -

        I just noticed that security_repository.xml is used at run-time. I propose moving it to META-INF/ojb/

        Show
        Michael Lipp added a comment - I just noticed that security_repository.xml is used at run-time. I propose moving it to META-INF/ojb/

          People

          • Assignee:
            David Le Strat
            Reporter:
            Michael Lipp
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development