Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12351 )
Change subject: KUDU-2411: Set SASL_PATH if needed when starting MiniCluster ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/12351/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/12351/1//COMMIT_MSG@7 PS1, Line 7: KUDU-2411: Set SASL_PATH if needed when starting MiniCluster > What is the control shell tool? Are you talking about the "kudu" tool? If s Yeah the "control shell" is thing that runs when you run `kudu test mini_cluster`. But I see your point: if you modified the tool to look for "../lib/sasl2" and set SASL_PATH if it exists, you'd be vulnerable to the security issue Todd raised. -- To view, visit http://gerrit.cloudera.org:8080/12351 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iaaeb30781f4483910c35a20c6d7c76f7f85aa4ce Gerrit-Change-Number: 12351 Gerrit-PatchSet: 1 Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Brian McDevitt <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Tue, 05 Feb 2019 00:36:11 +0000 Gerrit-HasComments: Yes
