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

Allow internal Certificate Authority #160

Merged
merged 3 commits into from
Dec 9, 2017
Merged

Conversation

tstone2077
Copy link
Contributor

If ca_trust_path is passed into winrm.Protocol, it is generally ignored. This change uses it as well as the possible request environment variables (REQUESTS_CA_BUNDLE and CURL_CA_BUNDLE) to trust a local CA.

@coveralls
Copy link

coveralls commented Mar 16, 2017

Coverage Status

Coverage decreased (-0.4%) to 67.456% when pulling 5b8592e on tstone2077:master into 09a81d5 on diyan:master.

6 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 67.456% when pulling 5b8592e on tstone2077:master into 09a81d5 on diyan:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 67.456% when pulling 5b8592e on tstone2077:master into 09a81d5 on diyan:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 67.456% when pulling 5b8592e on tstone2077:master into 09a81d5 on diyan:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 67.456% when pulling 5b8592e on tstone2077:master into 09a81d5 on diyan:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 67.456% when pulling 5b8592e on tstone2077:master into 09a81d5 on diyan:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) to 67.456% when pulling 5b8592e on tstone2077:master into 09a81d5 on diyan:master.

If ca_trust_path is passed into winrm.Protocol, it is generally ignored. This change uses it as well as the possible request environment variables (REQUESTS_CA_BUNDLE and CURL_CA_BUNDLE) to trust a local CA.
@coveralls
Copy link

coveralls commented May 24, 2017

Coverage Status

Coverage increased (+3.7%) to 71.598% when pulling f2efd94 on tstone2077:master into b1a54bc on diyan:master.

@norrlandskatten
Copy link

It would be a dream come true if this was implemented. I've successfully tested this solution with ansible 2.3.1.0 and python 2.7.9.

@nitzmahone nitzmahone merged commit fa259df into diyan:master Dec 9, 2017
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.6%) to 68.28% when pulling 833d9a7 on tstone2077:master into f2fae36 on diyan:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.6%) to 68.28% when pulling 833d9a7 on tstone2077:master into f2fae36 on diyan:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-3.6%) to 68.28% when pulling 833d9a7 on tstone2077:master into f2fae36 on diyan:master.

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

4 participants