massakam opened a new pull request #3024: Enable SSL of LibCurl linked to C++ 
library
URL: https://github.com/apache/pulsar/pull/3024
 
 
   ### Motivation
   
   Currently, the C++ client library included in the distributed RPM package 
can not perform https lookup.
   
   > 2018-11-20 21:17:36.708 ERROR HTTPLookupService:200 | Unable to load SSL 
engine as default, for url 
https://pulsar.xxx.xxx.yahoo.co.jp:8443/admin/persistent/prop/global/ns/dest/partitions
   
   In addition, access to the ZTS fails if using Athenz authentication.
   
   > 2018-11-20 21:15:59.937 ERROR ZTSClient:318 | Response failed for url 
https://zts.xxx.yahoo.co.jp:8443/zts/v1/domain/xxx.xxx.xxx.xxx/token. Error 
Code 1
   
   I think these are because SSL disabled libcurl is linked to the pulsar 
library.
   
   ### Modifications
   
   - Upgraded libcurl included in the docker images to 7.58.0. This is because 
we can not build curl using static openssl library in versions lower than 
7.58.0.
   https://github.com/curl/curl/pull/2234
   - Build curl with `--with-ssl` option.
   - Changed the order of the libraries to be linked so that curl is ahead of 
openssl.
   https://github.com/curl/curl/issues/2559
   
   ### Result
   
   HTTPS lookup and access to the ZTS will work properly.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to