Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14713
Change subject: [java] fixed typo in the connection negotiation code ...................................................................... [java] fixed typo in the connection negotiation code This patch fixes a typo in the connection negotiation code in the Java client. Prior to this fix, channel bindings were not verified during connection negotiation because the peer certificate was not set. In addition, I modified the error handing code in Negotiator.java to about connection negotiation upon receiving SSLPeerUnverifiedException due to the absence or the presence of invalid bindings. I also added a test to verify that Kudu Java client doesn't connect to a server which doesn't provide valid channel binding information during the connection negotiation phase. Kudos to Andy Singer for pointing to the typo! Change-Id: I7bfd428128e224f03901a6cd7b33283495a28d54 --- M java/kudu-client/src/main/java/org/apache/kudu/client/Connection.java M java/kudu-client/src/main/java/org/apache/kudu/client/Negotiator.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java M src/kudu/rpc/server_negotiation.cc 4 files changed, 92 insertions(+), 11 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/13/14713/1 -- To view, visit http://gerrit.cloudera.org:8080/14713 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I7bfd428128e224f03901a6cd7b33283495a28d54 Gerrit-Change-Number: 14713 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
