Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WFLY-9707] Remove the JASPI module from the feature pack as this is now in the core feature pack. #11474

Merged
merged 2 commits into from
Aug 14, 2018

Conversation

darranl
Copy link
Contributor

@darranl darranl commented Jul 31, 2018

⚠️ Do not Rebase this PR, the commits in this PR are contained in further topic branches. ⚠️

This pull request depends on the WildFly Core PR wildfly/wildfly-core#3407

Thanks for submitting your Pull Request!

Please make sure your PR meets the following requirements:

  • [ x ] Pull Request title is properly formatted: [WFLY-XYZ] Subject or WFLY-XYZ Subject
  • [ x ] Pull Request contains link to the JIRA issue(s)
  • [ x ] Pull Request contains description of the issue(s)

The JASPI module is required in WildFly Core for the Elytron subsystem to depend on it, this change is to remove it from WildFly as we do not need to define it twice.

  • [ x ] Pull Request does not include fixes for issues other than the main ticket
  • [ x ] Attached commits represent units of work and are properly formatted

For bigger changes, major and minor component upgrades make sure your PR also meets following requirements:

  • Pull Request requires a change to the documentation
  • Documentation have been updated accordingly
  • Tests were added to cover changes

For new features ensure as well:

  • Analysis was done
  • Test Plan has been done
  • Tests were verified in advance

If you are not an active contributor of the WildFly project you can request sponsorship by one of the members to help guide you through the process.

@kabir kabir added the core-upgrade-needed PR requires a wildfly-core change to be merged and integrated first label Aug 1, 2018
<dependency>
<groupId>org.jboss.spec.javax.security.auth.message</groupId>
<artifactId>jboss-jaspi-api_1.1_spec</artifactId>
</dependency>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should be also removed from servlet-galleon-pack/pom.xml

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note to self this has now been done

@kabir
Copy link
Contributor

kabir commented Aug 13, 2018

Retest this please

@darranl
Copy link
Contributor Author

darranl commented Aug 13, 2018

@kabir I think we are still missing the related core update for this one.

@jamezp
Copy link
Member

jamezp commented Aug 13, 2018

retest this please

@jamezp jamezp removed the core-upgrade-needed PR requires a wildfly-core change to be merged and integrated first label Aug 13, 2018
@kabir kabir merged commit 00587ab into wildfly:master Aug 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants