Skip to content

Conversation

bzoz
Copy link
Contributor

@bzoz bzoz commented Jul 21, 2020

Powershell defaults to tls 1.0 - This forces powershell to use tls 1.2
Tls 1.2 is supported in all current microsoft operating systems and
needed to download chocolatey.

Fixes: #33140
PR-URL: #33609
Reviewed-By: Bartosz Sosnowski [email protected]
Reviewed-By: Michael Dawson [email protected]
Reviewed-By: Tobias Nießen [email protected]

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

Powershell defaults to tls 1.0 - This forces powershell to use tls 1.2
Tls 1.2 is supported in all current microsoft operating systems and
needed to download chocolatey.

Fixes: nodejs#33140
PR-URL: nodejs#33609
Reviewed-By: Bartosz Sosnowski <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
@nodejs-github-bot nodejs-github-bot added install Issues and PRs related to the installers. tools Issues and PRs related to the tools directory. v14.x windows Issues and PRs related to the Windows platform. labels Jul 21, 2020
@bzoz
Copy link
Contributor Author

bzoz commented Jul 21, 2020

1 similar comment
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@addaleax
Copy link
Member

addaleax commented Aug 8, 2020

On the one hand, the commit message here should be fixed, on the other hand, it should probably also match the one on master – what do we usually do in that type of situation?

@addaleax addaleax mentioned this pull request Aug 10, 2020
@codebytere codebytere force-pushed the v14.x-staging branch 2 times, most recently from 92340c9 to b30b518 Compare August 11, 2020 16:39
MylesBorins pushed a commit that referenced this pull request Aug 17, 2020
Powershell defaults to tls 1.0 - This forces powershell to use tls 1.2
Tls 1.2 is supported in all current microsoft operating systems and
needed to download chocolatey.

Fixes: #33140
Backport-PR-URL: #34461
PR-URL: #33609
Reviewed-By: Bartosz Sosnowski <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
@MylesBorins
Copy link
Contributor

landed in aaa6e43

@addaleax I ended up keeping the original message to not mess up tooling. It is only on staging though, so we could add subsystem if we want

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

Labels

install Issues and PRs related to the installers. tools Issues and PRs related to the tools directory. windows Issues and PRs related to the Windows platform.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants