Skip to content

Commit

Permalink
Broker fails to start with function worker enabled and broker client …
Browse files Browse the repository at this point in the history
…using TLS (apache#5151)

* Fixing function worker running with broker when using TLS

* Get function worker TLS config from function_worker.yml
  • Loading branch information
cdbartholomew authored and merlimat committed Sep 24, 2019
1 parent 2ad58c4 commit 6741bfe
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@ private static class BrokerStarter {
workerConfig.setZooKeeperSessionTimeoutMillis(brokerConfig.getZooKeeperSessionTimeoutMillis());
workerConfig.setZooKeeperOperationTimeoutSeconds(brokerConfig.getZooKeeperOperationTimeoutSeconds());

workerConfig.setUseTls(brokerConfig.isTlsEnabled());
workerConfig.setUseTls(useTls);
workerConfig.setTlsHostnameVerificationEnable(false);

workerConfig.setTlsAllowInsecureConnection(brokerConfig.isTlsAllowInsecureConnection());
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -306,7 +306,7 @@ public void start() throws Exception {
workerConfig.setZooKeeperSessionTimeoutMillis(config.getZooKeeperSessionTimeoutMillis());
workerConfig.setZooKeeperOperationTimeoutSeconds(config.getZooKeeperOperationTimeoutSeconds());

workerConfig.setUseTls(config.isTlsEnabled());
workerConfig.setUseTls(useTls);
workerConfig.setTlsHostnameVerificationEnable(false);

workerConfig.setTlsAllowInsecureConnection(config.isTlsAllowInsecureConnection());
Expand Down

0 comments on commit 6741bfe

Please sign in to comment.