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

[ELY-2691] Merge if statement with the enclosing one in X509RevocationT… #2024

Merged
merged 1 commit into from
Oct 20, 2023

Conversation

gabrielpadilh4
Copy link
Contributor

Merge if statement with the enclosing one in X509RevocationTrustManager#checkCertPathLength

https://issues.redhat.com/browse/ELY-2691

Copy link
Contributor

@PrarthonaPaul PrarthonaPaul left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you @gabrielpadilh4

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.

Looks good, thanks @gabrielpadilh4!

…rustManager#checkCertPathLength

ELY-2691 Merge if statement with the enclosing one in X509RevocationTrustManager#checkCertPathLength
@gabrielpadilh4
Copy link
Contributor Author

Hello @fjuma fixed the trailing space that was failing on the pipeline.

@Skyllarr
Copy link
Contributor

@gabrielpadilh4 Thank you!

@Skyllarr Skyllarr merged commit 8f2e8fc into wildfly-security:2.x Oct 20, 2023
3 checks passed
@gabrielpadilh4 gabrielpadilh4 deleted the ELY-2691 branch October 20, 2023 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants