Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/19849


Change subject: [thirdparty] handle OpenSSL errors properly in curl and squeasel
......................................................................

[thirdparty] handle OpenSSL errors properly in curl and squeasel

This patch improves handling of OpenSSL errors in squeasel and curl,
making the code more robust.

This patch addresses the flakiness in the newly added JWT tests
that were failing in DEBUG-enabled builds because of the assertions
triggered by the SCOPED_OPENSSL_NO_PENDING_ERRORS macro.

I'm planning to submit the patches to the CURL project upstream and
post a PR to the squeasel repo [1] correspondingly.  To expedite the
fix and avoid too many unrelated test failures for every CR submitted
to the upstream gerrit, it makes sense to just patch the current source
of these two thirdparty components as an interim solution.

[1] https://github.com/cloudera/squeasel

Change-Id: I209ceb35ded66222c90b82e98e7d191f33573217
---
M thirdparty/download-thirdparty.sh
A thirdparty/patches/curl-handle-openssl-errors.patch
A thirdparty/patches/squeasel-handle-openssl-errors.patch
3 files changed, 320 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/49/19849/1
--
To view, visit http://gerrit.cloudera.org:8080/19849
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I209ceb35ded66222c90b82e98e7d191f33573217
Gerrit-Change-Number: 19849
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to