-
Notifications
You must be signed in to change notification settings - Fork 765
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
Dynamically set TLS/SSL certificate path #2320
Comments
|
Curl and CA bundle was/is a recurring annoyance in GDAL too. |
|
One option worth considering too (not ideal, but can help when other options fail) is to offer the possibility to disable certificate checks: https://github.com/OSGeo/gdal/blob/master/gdal/port/cpl_http.cpp#L1797 |
|
I see it uses: https://curl.haxx.se/libcurl/c/CURLOPT_CAINFO.html, that is good to know. I propose to have PROJ to have this CA Bundle search logic to set
Sound like a good idea?
I see the |
When building wheels for
pyprojI have discovered that includingcurlis not enough. I am having issues using PROJ_NETWORK since the path to the CA bundle can vary depending on the operating system/virtual environment.Here is the error I am running into:
One idea I have to ensure that users have an up-to-date CA bundle would be to use the python
certifilibrary. Doing so would require the capability to dynamically set the path to the CA bundle. I attempted to modify theCURL_CA_BUNDLEenvironment variable and it worked on OSX, but I have not got it working on Linux (yet). It would be nice to be able to scope this so you only set it for thecurlused byPROJ.Curl has this method that can be used to dynamically set the path & type:
This is something that will need to be updated in
networkfilemanager.cpp. I am thinking adding a method to set this would be useful (const int proj_curl_set_sslcert(const char*cert_path, const char* cert_type)).Thoughts? Suggestions?
The text was updated successfully, but these errors were encountered: