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 2173] Add test for the CLIENT_CERT mechanism #1955
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the contribution, I would just mention a couple of the new files are missing the standard copyright header we use in the project if you are able to please add them.
@darranl Thanks for the correction. I've also rebased the branch to the latest 2.x changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @Dkafetzis looks like the CI run was green as well.
tests/base/src/test/java/org/wildfly/security/http/impl/AbstractBaseHttpTest.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Dkafetzis! Have added some small comments.
tests/base/src/test/java/org/wildfly/security/http/impl/AbstractBaseHttpTest.java
Outdated
Show resolved
Hide resolved
@fjuma Could I get a review on this again? |
tests/base/src/test/java/org/wildfly/security/http/impl/AbstractBaseHttpTest.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Dkafetzis!
https://issues.redhat.com/browse/ELY-2173