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

ELY-2638 : Simplify test assertions in SSLAuthenticationTest #2013

Merged
merged 1 commit into from Mar 14, 2024
Merged

ELY-2638 : Simplify test assertions in SSLAuthenticationTest #2013

merged 1 commit into from Mar 14, 2024

Conversation

PrarthonaPaul
Copy link
Contributor

@PrarthonaPaul PrarthonaPaul mentioned this pull request Oct 16, 2023
Closed
@fjuma
Copy link
Contributor

fjuma commented Mar 14, 2024

@PrarthonaPaul Whenever you get a chance, would you be able to rebase this to re-trigger CI? Thanks!

@PrarthonaPaul
Copy link
Contributor Author

@PrarthonaPaul Whenever you get a chance, would you be able to rebase this to re-trigger CI? Thanks!

Done!

fjuma
fjuma approved these changes Mar 14, 2024
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 @PrarthonaPaul!

@fjuma fjuma added +1 FJ and removed Needs CI labels Mar 14, 2024
@fjuma fjuma merged commit d530677 into wildfly-security:2.x Mar 14, 2024
3 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

@fjuma fjuma fjuma approved these changes

@Skyllarr Skyllarr Awaiting requested review from Skyllarr Skyllarr is a code owner

Assignees
No one assigned
Labels
+1 DAL +1 FJ
Projects
None yet
Milestone
No milestone
3 participants
@PrarthonaPaul @fjuma @darranl