Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/19799
Change subject: [util] fix non-handled OpenSSL errors in JWT code ...................................................................... [util] fix non-handled OpenSSL errors in JWT code While looking at ~50% flakiness stats for the newly added JWT test, I found the root cause of the issue was a non-handled error left at the error stack of the OpenSSL library. Further, while looking at the JWT code, I found that the code that uses OpenSSL API was fragile and corresponding errors were either not handled at all or handled improperly. While this patch doesn't fix the flakiness in the bespoken JWT test, it addresses the issue with the funny code in the JWT wrapper where OpenSSL errors were not handled. Change-Id: Iac5142c1dcaeec5042217e1c90f8e8770b36a670 --- M src/kudu/util/jwt-util-internal.h M src/kudu/util/jwt-util.cc 2 files changed, 124 insertions(+), 81 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/99/19799/1 -- To view, visit http://gerrit.cloudera.org:8080/19799 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Iac5142c1dcaeec5042217e1c90f8e8770b36a670 Gerrit-Change-Number: 19799 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
