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-5566] Use com.sun.net.ssl.internal.ssl.Provider.isFIPS() method for FIPS detection if it exists #4849

Merged
merged 1 commit into from
Nov 10, 2021
Merged

[WFCORE-5566] Use com.sun.net.ssl.internal.ssl.Provider.isFIPS() method for FIPS detection if it exists #4849

merged 1 commit into from
Nov 10, 2021

Conversation

Skyllarr
Copy link
Contributor

@Skyllarr Skyllarr commented Nov 3, 2021

@github-actions github-actions bot added the deps-ok Dependencies have been checked, and there are no significant changes label Nov 3, 2021
@jmesnil jmesnil requested a review from fjuma November 8, 2021 08:49
fjuma
fjuma approved these changes Nov 8, 2021
View reviewed changes
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 @Skyllarr!

@fjuma
Copy link
Contributor

fjuma commented Nov 8, 2021

@OndrejKotek Did you want to take a look at this one?

@OndrejKotek
Copy link
Contributor

@fjuma I have had a look and performed several tests and it seems to work as expected, thanks.

@jmesnil jmesnil added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Nov 10, 2021
@jmesnil jmesnil merged commit a19937e into wildfly:main Nov 10, 2021
@jmesnil
Copy link
Member

jmesnil commented Nov 10, 2021

@Skyllarr thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fjuma fjuma fjuma approved these changes

Assignees
No one assigned
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
Milestone
No milestone
4 participants
@Skyllarr @fjuma @OndrejKotek @jmesnil