Uploaded image for project: 'WildFly Core'
  1. WildFly Core
  2. WFCORE-4640

Add a DUP to the Elytron subsystem that adds the 'javax.security.auth.message.api' and 'javax.security.jacc.api' modules to the deployment

XMLWordPrintable

      Discussion from chat:

      Brian Stansberry: @Farah Juma Should the elytron subsystem have a DUP that does this:

      Brian Stansberry: https://github.com/wildfly/wildfly/blob/master/security/subsystem/src/main/java/org/jboss/as/security/deployment/SecurityDependencyProcessor.java#L68

      Brian Stansberry: that module is a dep of the subsystem, so it is available

      Brian Stansberry: background – Jean-Francois and I are checking through various Galleon layers use cases (hence the JSR375 thread) and it struck me that we shouldn't need legacy security anymore in ts/integration/elytron

      Brian Stansberry: i.e. a PR like this should work:

      Brian Stansberry: https://github.com/wildfly/wildfly/compare/master...bstansberry:elytron-no-legacy

      Brian Stansberry: it mostly does, e.g. https://ci.wildfly.org/viewLog.html?buildId=164893&buildTypeId=WF_MasterLinuxJdk11

      Brian Stansberry: that DynamicJaspiTestCase failure I'm guessing is because without the legacy security subsystem we have no DUP making the jaspi module available to the deployment

      Farah Juma: @Brian Stansberry Ah, interesting. It seems like it would make sense for the Elytron subsystem to have a DUP that adds both that one and the JACC one

      Brian Stansberry: yeah

      Farah Juma: I can create a PR for that and then we can check with @Darran Lofthouse to make sure that's the right thing to do

      Brian Stansberry: actually, that seems pretty critical, as the layers we want to support on OpenShift do not have the legacy security subsystem

      Brian Stansberry: if you can do that i'd appreciate it very much

      Farah Juma: sure

              fjuma1@redhat.com Farah Juma
              fjuma1@redhat.com Farah Juma
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: