-
Notifications
You must be signed in to change notification settings - Fork 943
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
Openssl support #31
Comments
Is this OpenSSL specific or are you talking about generic TLS support? I guess as user of curl, you should be able to abstract away from the TLS implementation used. |
Yeah I should clarify that I want generic TLS support. The library client should probably not care what's used behind the scenes to do TLS, this is purely a build step detail. That said, if I were to embed a particular TLS library, OpenSSL appears to be the way to go. |
On Windows and OS X, you can configure cURL to simply use the OS-provided TLS stacks (Secure Channel, Secure Transport), which has multiple upsides to shipping OpenSSL or one of its forks:
On Linux, you can feed the OS-provided trusted root certificates to cURL if you can find them. Here's where we look for them in Kullo: The CA bundle can then be fed to cURL using |
FYI for people just looking to get a quick and dirty requests over SSL should be able to do so by linking against libCURL (compiled with OpenSSL). |
is it currently possible to make connections via Oauth? |
There are some options for SSL: Not check SSL:
Self-signed serfiticate:
Signed sertificate: I't good to use public web servers for unit tests. |
I disagree: you want unit tests to be able to run without any external dependencies. Integration tests are a different story though. But even those you probably want to run against a server which configuration you control (.e.g one you spawn as part of the test run). |
Btw, couldn't find in the docs but there is a way to disable SSL checks without going down to libcurl calls.
Hope it helps someone! |
Thank you so much! Simple and effective! |
What's the status of this? It looks like the CI has fallen behind. However the merge isn't all that big to begin with. What's holding this feature back from being merged into mainline? |
+1 |
1 similar comment
+1 |
No description provided.
The text was updated successfully, but these errors were encountered: