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.

Sign up for GitHub

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

Jump to bottom

[WFCORE-6352] Add the ability to secure the management interface using a virtual HTTP authentication factory if available #5509

Merged
merged 1 commit into from
May 26, 2023
Merged

[WFCORE-6352] Add the ability to secure the management interface using a virtual HTTP authentication factory if available #5509

merged 1 commit into from
May 26, 2023

Conversation

fjuma
Copy link
Contributor

@fjuma fjuma commented May 18, 2023

@fjuma
[WFCORE-6352] Add the ability to secure the management interface usin…
38cb5b3 
…g a virtual HTTP authentication factory if available
@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label May 18, 2023
@wildfly-ci
Copy link

Core -> Full Integration Build 12343 outcome was FAILURE using a merge of 38cb5b3
Summary: Tests passed: 1694, ignored: 31; exit code 1 (Step: Build & test full (Maven)) Build time: 00:35:23

@wildfly-ci
Copy link

Core -> Full Integration Build 12344 outcome was FAILURE using a merge of 38cb5b3
Summary: Tests passed: 1694, ignored: 31; exit code 1 (Step: Build & test full (Maven)) Build time: 00:36:01

@fjuma fjuma mentioned this pull request May 18, 2023
Merged
@wildfly-ci
Copy link

Core -> Full Integration Build 12345 outcome was FAILURE using a merge of 38cb5b3
Summary: Tests passed: 1694, ignored: 31; exit code 1 (Step: Build & test full (Maven)) Build time: 00:49:15

@fjuma fjuma requested review from darranl and Skyllarr May 19, 2023 20:42
@fjuma fjuma marked this pull request as ready for review May 23, 2023 20:16
@darranl darranl mentioned this pull request May 24, 2023
Closed
@yersan yersan added ready-for-merge This PR is ready to be merged and fulfills all requirements and removed ready-for-merge This PR is ready to be merged and fulfills all requirements labels May 25, 2023
@darranl
Copy link
Contributor

darranl commented May 25, 2023

Please don't make any further changes to this PR - I am going to combine it with a couple of other PRs so we can get them merged together.

Thank you for this implementation.

@darranl darranl added the ready-for-merge This PR is ready to be merged and fulfills all requirements label May 25, 2023
@darranl darranl mentioned this pull request May 25, 2023
Merged
@darranl darranl merged commit 97e0d7f into wildfly:main May 26, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@darranl darranl darranl approved these changes

@Skyllarr Skyllarr Skyllarr approved these changes

Assignees
No one assigned
Labels
21.0.0.Beta1 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
Milestone
No milestone
5 participants
@fjuma @wildfly-ci @darranl @Skyllarr @yersan