Uh oh!
There was an error while loading. Please reload this page.
HBASE-27342 Use Hadoop Credentials API to retrieve passwords of TLS key/trust stores - #4747
Conversation
Apache-HBase
commented
Aug 30, 2022
💔 -1 overall
This message was automatically generated. |
meszibalu
left a comment
There was a problem hiding this comment.
I added only two small comments. They weren't introduced in this commit, I have just noticed it.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Apache-HBase
commented
Aug 30, 2022
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Aug 30, 2022
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Aug 30, 2022
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Aug 30, 2022
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Aug 31, 2022
🎊 +1 overall
This message was automatically generated. |
Apache9
left a comment
There was a problem hiding this comment.
The CredentialProviderFactory and CredentialProvider is IA.Public so I think it is OK to use it directly. And this is the CredentialProvider can be inherited and customized which is also good for users who have their own way to store password.
+1
Apache-HBase
commented
Aug 31, 2022
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Aug 31, 2022
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Aug 31, 2022
🎊 +1 overall
This message was automatically generated. |
…ey/trust stores (apache#4747) Signed-off-by: Andrew Purtell <apurtell@apache.org> Signed-off-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Balazs Meszaros <meszibalu@apache.org>
cc @bbeaudreault@Apache9@apurtell