There was a problem with the code just a little while ago where specifying an HTTPS URL as the serviceUrl caused a segfault. This has been fixed by https://github.com/apache/pulsar-client-node/pull/310, but since there were no tests for this issue yet, I added it.
In addition, it seemed that the place where -fvisibility=hidden should be added was wrong, so I fixed it.
Verifying this change
[x] Make sure that the change passes the CI checks.
Documentation
[ ] doc-required
(Your PR needs to update docs and you will update later)
Motivation
There was a problem with the code just a little while ago where specifying an HTTPS URL as the
serviceUrlcaused a segfault. This has been fixed by https://github.com/apache/pulsar-client-node/pull/310, but since there were no tests for this issue yet, I added it.In addition, it seemed that the place where
-fvisibility=hiddenshould be added was wrong, so I fixed it.Verifying this change
Documentation
[ ]
doc-required(Your PR needs to update docs and you will update later)[x]
doc-not-needed(Please explain why)[ ]
doc(Your PR contains doc changes)[ ]
doc-complete(Docs have been already added)