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-2271 IdentityCredentials.verify(Supplier<Provider[]>, evidence, h… #1638
Conversation
Linux - JDK11 EA 28 Build 834 outcome was FAILURE using a merge of 9b9a82d Failed tests
|
Windows Build 814 outcome was FAILURE using a merge of 9b9a82d Failed tests
|
0daff58
to
041ce47
Compare
18d0c1a
to
4c80b6f
Compare
/** | ||
* Creates a new instance. | ||
* | ||
* @param realm the {@link CacheableSecurityRealm} whose {@link RealmIdentity} should be cached.. |
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.
Just a small comment, there's an extra period at the end of this line
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.
FIxed this one and the one above
@@ -45,7 +45,7 @@ | |||
public BasicMechanismFactory() { | |||
} | |||
|
|||
public BasicMechanismFactory(final Provider provider) { | |||
public BasicMechanismFactory(final Provider... providers) { |
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.
Just FYI for other reviewers, this isn't public API.
@tristantarrant This looks good to me. Just noticed, this PR was submitted against the 1.x branch. If you'd like to get this fix into 1.17.x, you can update this to be directly against that branch. We then forward port to 1.x as part of our release process. |
4c80b6f
to
f8f3190
Compare
* Deprecate methods which use the installed providers
f8f3190
to
92a87ff
Compare
Thanks, @tristantarrant! I've now created the 1.17.x branch. If you could update this PR to be based off that branch that would be great. (I tried changing the base branch myself but more work is needed.) |
1.17 backport: #1641 |
Perfect, thanks @tristantarrant! I'm going to close this one now in favour of #1641. We forward port the changes from maintenance branches to 1.x as part of our release process so we don't need the additional PR. |
https://issues.redhat.com/browse/ELY-2271
https://issues.redhat.com/browse/ELY-2272
Please note the deprecations I've added.