vikaskr22 commented on code in PR #723:
URL: https://github.com/apache/ranger/pull/723#discussion_r2509961418
##########
agents-common/src/main/java/org/apache/ranger/plugin/util/PasswordUtils.java:
##########
@@ -67,12 +66,12 @@ public class PasswordUtils {
if (cryptAlgoArray != null && cryptAlgoArray.length > 4) {
int index = 0;
- cryptAlgo = cryptAlgoArray[index++]; // 0
+ cryptAlgo =
RangerSupportedCryptoAlgo.valueOf(cryptAlgoArray[index++]); // 0
Review Comment:
Initially idea was to keep all supported algorithm at one place. Although I
have put many used PBE based algorithm in the enum, But I agree that someone
might might have used a totally different crypto algorithm.
I am working to update the code to support this behaviour as well. Can we
merge this PR and follow this as a separate JIRA ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
