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

Add --insecure option for SSL curl requests. #805

Merged
merged 1 commit into from
Jun 19, 2017

Conversation

tvandijck
Copy link
Contributor

We run into a lot of ssl errors when communicating with the outside world due to proxy settings.
This allows us to run some of the nuget tests without having to locally modify the code.

Copy link
Member

@samsinsane samsinsane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While adding this is good from a completeness point of view, I feel like this is something you never want to actually use. (For security reasons)

Would you be able to solve your problem with the curl proxy options?

if http ~= nil then
newoption {
trigger = "insecure",
description = "forfit SSH certification checks."
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor typo, "forfeit".

@tvandijck
Copy link
Contributor Author

I feel like this is something you never want to actually use.

I agree, that in general you do never want to use it... However, there is cases where a company for all kinds of reasons I don't agree with, puts in a proxy server that rewrites all certificates, it's basically an IT endorsed man in the middle attack. For example to monitor employees leaking sensitive data, or for whatever other reason... The two options I'm disabling here:

		curl_easy_setopt(curl, CURLOPT_SSL_VERIFYHOST, 0);
 		curl_easy_setopt(curl, CURLOPT_SSL_VERIFYPEER, 0);

All nuget unit-tests here fail when executed on my desktop without this option.

@samsinsane
Copy link
Member

My understanding of proxies for HTTPS is rather limited, but what about options such as CURLOPT_PROXY_SSLCERT? I'm happy for this to go through, I'm just hoping that curl supports security, even if it's by allowing a specific MitM attack rather than all of them.

@tvandijck tvandijck merged commit 8d509c6 into premake:master Jun 19, 2017
@tvandijck tvandijck deleted the insecure-option branch June 19, 2017 20:06
@tvandijck
Copy link
Contributor Author

@samsinsane I looked into that, didn't get much results... it's weird too though, I only get this on my desktop machine, but not on our jenkins boxes...

I'm merging it, but will put this on my todo list to investigate further.

@samsinsane
Copy link
Member

@tvandijck That's fair enough, I noticed those options when I added the proxy URL and just figured that this might be what they're for? Ah well, if it's what you need to get through your proxy server, then I imagine others will need it too.

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.

2 participants