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

curl needs -k otherwise it fails with cert warning #441

Closed
wants to merge 1 commit into from
Closed

curl needs -k otherwise it fails with cert warning #441

wants to merge 1 commit into from

Conversation

stmuk
Copy link

@stmuk stmuk commented Apr 9, 2015

No description provided.

@miyagawa
Copy link
Owner

miyagawa commented Apr 9, 2015

Nope, your curl is old and needs to be updated.

@miyagawa miyagawa closed this Apr 9, 2015
@stmuk
Copy link
Author

stmuk commented Apr 9, 2015

True but your documentation still will fail for 90% of the people reading it.
And the point of documentation is to help people.

@virtualsue
Copy link

It has always failed for me. This documentation needs to be updated. From the usual workflow, people are not going to know that they "need" to update their version of curl so that your instructions work. And in fact their version of curl works just fine.

@miyagawa
Copy link
Owner

miyagawa commented Apr 9, 2015

@stmuk where does the number "90%" come from? Do you have statistics of the versions of curl people are using? I suggest you not to try to convince me by making up numbers.

@virtualsue their curl doesn't work "just fine" - given the number of SSL vulnerabilities for the recent years, using -k is the most insecure thing. http://curl.haxx.se/docs/sslcerts.html

For me, this option has been working for the past 5 years (when cpanm was first released in 2010) with curl that comes with the stock OS X.

I know there are systems that are shipped with either old curl or outdated CA certificates, but enabling -k for these systems is a step backwards. I'd accept the patch that says along the lines of "if you receive a warning from curl, you're strongly suggested to update the version of curl to support the newer certificates." etc.

@miyagawa
Copy link
Owner

addressed with 2b1fa01

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

Successfully merging this pull request may close these issues.

None yet

3 participants