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

[WFCORE-6823] Don't register the Jakarta AuthConfigFactory in the host controller. #5993

Merged
merged 1 commit into from
May 20, 2024

Conversation

darranl
Copy link
Contributor

@darranl darranl commented May 9, 2024

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label May 9, 2024
@yersan
Copy link
Collaborator

yersan commented May 10, 2024

@darranl just asking, shouldn't this change be driven by the RFE that would move Jakarta Authentication to its own subsystem?

@darranl
Copy link
Contributor Author

darranl commented May 15, 2024

The reason I am submitting this now is so the pre-requisite changes can come through without getting to the point that everything needs coordinated tag and merge activity.

IMO this is an internal refactoring and is not by itself feature development and is safe to be merged independently.

@darranl
Copy link
Contributor Author

darranl commented May 16, 2024

@yersan Actually ignore my last comment, this is just a bug fix for a bug - this is not a part of any feature development.

@bstansberry
Copy link
Contributor

@darranl The JIRA description could use an update then, as the 'bug' aspect is unclear.

@darranl
Copy link
Contributor Author

darranl commented May 17, 2024

@yersan / @bstansberry I have updated the description in the issue to say it is to fix the bug,

@yersan yersan added the ready-for-merge This PR is ready to be merged and fulfills all requirements label May 20, 2024
@yersan yersan merged commit fe884ea into wildfly:main May 20, 2024
12 checks passed
@yersan
Copy link
Collaborator

yersan commented May 20, 2024

Thanks @darranl @bstansberry

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 ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants