impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Ho (Code Review)" <>
Subject [Impala-ASF-CR] KUDU-1929: [rpc] Allow using encrypted private keys for TLS
Date Sun, 20 Aug 2017 01:11:40 GMT
Hello Sailesh Mukil, Kudu Jenkins,

I'd like you to do a code review.  Please visit

to review the following change.

Change subject: KUDU-1929: [rpc] Allow using encrypted private keys for TLS

KUDU-1929: [rpc] Allow using encrypted private keys for TLS

* This patch adds a new flag for a "password command" for the RPC
  private key.

 * This also makes the webserver move to using the new function

 * This also consolidates certificates from security-test-util into

Testing: Adds 2 tests. One to verify that RPCs work when providing
the right password for password protected private keys, and one to
verify that the Messenger does not startup if the wrong password
is provided when using a password protected private key.

Change-Id: Ifd6369581fa426ceab11e4a10441658c7da47e81
Tested-by: Kudu Jenkins
Reviewed-by: Sailesh Mukil <>
M be/src/kudu/rpc/
M be/src/kudu/rpc/
M be/src/kudu/security/
M be/src/kudu/security/crypto.h
M be/src/kudu/security/
M be/src/kudu/security/openssl_util.h
M be/src/kudu/security/openssl_util_bio.h
M be/src/kudu/security/
M be/src/kudu/security/security-test-util.h
M be/src/kudu/security/test/
M be/src/kudu/security/test/test_certs.h
M be/src/kudu/security/
M be/src/kudu/security/tls_context.h
13 files changed, 329 insertions(+), 144 deletions(-)

  git pull ssh:// refs/changes/41/7741/1
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifd6369581fa426ceab11e4a10441658c7da47e81
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Michael Ho <>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Sailesh Mukil <>

View raw message