Skip to content
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

Support SSL related configuration #6

Closed
carllerche opened this issue Jul 2, 2014 · 4 comments · Fixed by #105
Closed

Support SSL related configuration #6

carllerche opened this issue Jul 2, 2014 · 4 comments · Fixed by #105

Comments

@carllerche
Copy link
Contributor

Allow configuring CURL's SSL related settings.

@insectatorious
Copy link

Can we set CURLOPT_SSL_VERIFYPEER to false ourselves somehow until this is exposed? Looking tenatively at curl-sys for inspiration.

@posix4e
Copy link

posix4e commented Jun 5, 2015

I'm curious what we are looking for here. What ssl capability are we looking for.

@insectatorious
Copy link

Doc for ref.

I would be using this to make requests against a test environment where we have an in-house SSL certificate. In this case disabling the verification is not necessarily a bad thing (at least this is the easiest approach but may not be the most correct way).

Anyway, seeing as CURL supports this I thought this should as well.

@posix4e
Copy link

posix4e commented Jun 5, 2015

It's easier for me the more fine gained issues are. If it's just that one
thing let's change the title.

On Fri, Jun 5, 2015, 00:14 Sumanas Sarma notifications@github.com wrote:

Doc for ref http://curl.haxx.se/libcurl/c/CURLOPT_SSL_VERIFYPEER.html.

I would be using this to make requests against a test environment where we
have an in-house SSL certificate. In this case disabling the verification
is not necessarily a bad thing (at least this is the easiest approach but
may not be the most correct way).

Anyway, seeing as CURL supports this I thought this should as well.


Reply to this email directly or view it on GitHub
#6 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants