Skip to content
This repository has been archived by the owner on Jun 7, 2021. It is now read-only.

SWARM-582: saaj-impl module not found #162

Closed
wants to merge 1 commit into from
Closed

SWARM-582: saaj-impl module not found #162

wants to merge 1 commit into from

Conversation

kenfinnigan
Copy link
Member

  • Have you followed the guidelines in our Contributing document?
  • Have you created a JIRA and used it in the commit message?
  • Have you checked to ensure there aren't other open Pull Requests for the same issue?
  • Have you built the project locally prior to submission with mvn clean install?

Motivation

Within the webservices subsystem of WildFly it directly loads this module when it's unable to find a service entry for javax.xml.soap.MetaFactory.

As its directly loading it, our plugin doesn't have a way to know it needs that module present so we need to specifically reference it in our build.

Modifications

Added a test showing the problem and then updated module.conf for webservices fraction to include saaj-impl

Result

Solves exception when trying to lookup a MetaFactory for SOAP.

Motivation
----------
Within the webservices subsystem of WildFly it directly loads this module when it's unable to find a service entry for javax.xml.soap.MetaFactory.

As its directly loading it, our plugin doesn't have a way to know it needs that module present so we need to specifically reference it in our build.

Modifications
-------------
Added a test showing the problem and then updated module.conf for webservices fraction to include saaj-impl

Result
------
Solves exception when trying to lookup a MetaFactory for SOAP.
@kenfinnigan
Copy link
Member Author

closing in favor of #164

@kenfinnigan kenfinnigan deleted the SWARM-582 branch September 30, 2016 20:16
Sanne pushed a commit to Sanne/thorntail that referenced this pull request Mar 3, 2017
* Do not mention wildfly-swarm-bootstrap in the bootstrap.conf
* Export the org.wildfly.swarm.bootstrap.util.* as part of org.wildfly.swarm.bootstrap:main
* Have container:api import org.wildfly.swarm.bootstrap:main so it can use WildFlySwarmBootstrapConf class.
antoinesd pushed a commit to antoinesd/thorntail that referenced this pull request Apr 3, 2019
Changes to improve PME alignments with 2.3.0
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
1 participant