Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/19617 )
Change subject: KUDU-3448 Add support for encrypting TSKs ...................................................................... Patch Set 7: (1 comment) http://gerrit.cloudera.org:8080/#/c/19617/6/src/kudu/security/token_signing_key.cc File src/kudu/security/token_signing_key.cc: http://gerrit.cloudera.org:8080/#/c/19617/6/src/kudu/security/token_signing_key.cc@98 PS6, Line 98: CHECK_OK(key_->ToEncryptedString(&private_key_der_, DataFormat::DER, : [&](string* password){ : RETURN_NOT_OK_PREPEND(GetPasswordFromShellCommand( : FLAGS_tsk_private_key_password_cmd, password), : "could not get TSK private key password from configured command"); : return Status::OK(); : } : )); > Thanks! Posted a follow-up patch to address this issue. -- To view, visit http://gerrit.cloudera.org:8080/19617 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Id8d770de7ed824cfc725003bbe77f1e42629029b Gerrit-Change-Number: 19617 Gerrit-PatchSet: 7 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Reviewer: Ádám Bakai <[email protected]> Gerrit-Comment-Date: Thu, 11 May 2023 15:38:46 +0000 Gerrit-HasComments: Yes
