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-15582] Remove unneeded code from legacy security subsystem; prune… #14832

Merged
merged 1 commit into from Nov 6, 2021

Conversation

bstansberry
Copy link
Contributor

@bstansberry bstansberry commented Oct 24, 2021

… pom and module dependencies

https://issues.redhat.com/browse/WFLY-15582

First bit of work for WFLY-15566, which is to restore the legacy security extension to WildFly Preview, as there's no reason not to make its current model-only incarnation available there.

But first I want to reduce the module dependencies to the needed minimum. Which led to a general code deletion and tightening exercise. That seems worthwhile on its own and need not block for the WF Preview work, so here it is.

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Oct 24, 2021
@bstansberry bstansberry force-pushed the leg_sec_clean branch 3 times, most recently from bab318f to bbc8686 Compare October 25, 2021 17:51
…ne pom and module dependencies

This also resulted in quite a few tweaks to the integration testsuite, primarily to poms, to deal with dependencies that were being brought in transitively via the legacy security subsystem module and are no longer used there.
@bstansberry bstansberry changed the title [WFLY-TBD] Remove unneeded code from legacy security subsystem; prune… [WFLY-15582] Remove unneeded code from legacy security subsystem; prune… Oct 28, 2021
@bstansberry bstansberry marked this pull request as ready for review October 28, 2021 11:48
@bstansberry
Copy link
Contributor Author

@fjuma Please review.

Copy link
Contributor

@fjuma fjuma left a comment

Choose a reason for hiding this comment

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

Thanks @bstansberry!

@bstansberry bstansberry merged commit 21aacd5 into wildfly:main Nov 6, 2021
@bstansberry bstansberry deleted the leg_sec_clean branch November 6, 2021 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes
Projects
None yet
2 participants