-
-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix detection of usable CA certificate bundle #45
Conversation
Fixes segfaults in #34.
e7c0ca0
to
b8883c9
Compare
On FreeBSD, we now get
|
Works fine on helloSystem in my test. Are you sure you used the latest binaries? |
Hold on... retesting. Edit:
Apparently I didn't, see below. Sorry for the noise. |
FYI, I want to improve the history, so please leave the merge to me once you approve the changes. I'll take care of it. |
Can confirm that the latest build works on |
Also confirm that the latest build works on |
When the values libcurl provides as defaults are broken, they should be unset or replaced with working ones. Therefore, we now probe them, unset broken ones and then run our detection logic if the values have not been set. This should make sure both values are either set with a seemingly usable path or unset. When neither of the values work, we log a warning. Then, the included locations can be amended, if possible.
Fixes segfaults in #34.
Also pins Alpine Docker image version to fix build issues due to the previous use of the bleeding
edge
tag.