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
[WFLY-13625] Security realms should support specifying the charset and encoding for credentials. #13548
Conversation
5d255f1
to
4d8a881
Compare
745e6f5
to
c6ef824
Compare
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 have a small documentation comment but the rest looks good to me. Am going to mark this one as approved.
|
||
The various password mappers allow loading multiples values from the database to reconstruct the password. | ||
This information is then used to hash the client provided password to compare against the password reconstructed from the | ||
database. |
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.
Maybe this could be reworded to something like:
The various password mappers allow loading multiples values from the database to hash the client provided password in order to compare against the password stored in the database.
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!
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 @SoniaZaldana.
@SoniaZaldana as discussed we have run into some failures with the new FilesystemRealmTestCase, we should be ready for you to rebase on master now to reproduce locally. |
…d encoding for credentials.
Thanks @darranl, I fixed the test failure and rebased. |
Thank you @SoniaZaldana CI was clean. |
Jira: https://issues.redhat.com/browse/WFLY-13625
Requires: https://issues.redhat.com/browse/WFCORE-5027
WildFly Core PR with tests: wildfly/wildfly-core#4324