Skip to content
This repository has been archived by the owner on Oct 24, 2024. It is now read-only.

Allow enabled SSL protocols to be specified #86

Merged
merged 6 commits into from
Dec 25, 2018

Conversation

aadamovich
Copy link
Contributor

Pull Request (PR) description

When using download_file on some Windows systems against files on GitHub, I received:

  • FullyQualifiedErrorId : The request was aborted: Could not create SSL/TLS secure channel.

It appears that system didn't have all the SSL protocols enabled on group policy level, which I cannot change at the moment.

This pull request creates a workaround allowing to specify which protocol to use when downloading files.

Example:

  download_file { 'Download git':
    url                   => 'https://github.com/git-for-windows/git/releases/download/v2.19.2.windows.1/Git-2.19.2-64-bit.exe',
    security_protocol     => 'tls12',
    destination_directory => 'C:/software',
  }

manifests/init.pp Outdated Show resolved Hide resolved
@bastelfreak bastelfreak added the needs-work not ready to merge just yet label Dec 15, 2018
@bastelfreak
Copy link
Member

Hi @aadamovich, thanks for the PR. Can you please squash the commits into one and take a look at the inline comment I made?

@aadamovich
Copy link
Contributor Author

Hi @bastelfreak, I have changed the pp file according to your comments. Regarding squashing, isn't that something that GitHub supports when you press merge button? https://blog.github.com/2016-04-01-squash-your-commits/

@juniorsysadmin juniorsysadmin changed the title Powershell sometimes complains about not enabled SSL protocols Allow enabled SSL protocols to be specified Dec 25, 2018
@juniorsysadmin juniorsysadmin added the enhancement New feature or request label Dec 25, 2018
@bastelfreak
Copy link
Member

Hi @aadamovich, we always prefer it when the contributors provide a sane git history. Squashing all commits into a single one isn't the best solution for most situations.

@bastelfreak bastelfreak merged commit 9e6c642 into voxpupuli:master Dec 25, 2018
@bastelfreak bastelfreak removed the needs-work not ready to merge just yet label Dec 25, 2018
@alexjfisher
Copy link
Member

@bastelfreak Did this supercede #68 ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants