Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20093 )
Change subject: [util] KUDU-3392: add validator for --trusted_certificate_file ...................................................................... [util] KUDU-3392: add validator for --trusted_certificate_file While testing/troubleshooting the newly introduced JWT-based client authentication, I found that setting the flag to a non-existing file hadn't been handled as I expected. This patch addresses the issue, adding a validator for the flag. I didn't add any test to cover the newly added validator, but I have verified that it worked as expected in various scenarios w.r.t. the current value of the --trusted_certificate_file flag. This is a follow-up to 152211658ef9d33e0ad727ccba46f8af24cd45b0. Change-Id: I0d9d816a821a93037293d3985a2f577711d64ef2 Reviewed-on: http://gerrit.cloudera.org:8080/20075 Tested-by: Kudu Jenkins Reviewed-by: Attila Bukor <abu...@apache.org> (cherry picked from commit 4734d6f167333e67483d45bbeaf7509180641414) Reviewed-on: http://gerrit.cloudera.org:8080/20093 Reviewed-by: Yingchun Lai <laiyingc...@apache.org> --- M src/kudu/util/curl_util.cc 1 file changed, 44 insertions(+), 2 deletions(-) Approvals: Kudu Jenkins: Verified Yingchun Lai: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/20093 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: I0d9d816a821a93037293d3985a2f577711d64ef2 Gerrit-Change-Number: 20093 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>