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.

Sign up for GitHub

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

Jump to bottom

Switch to TLS 1.2 as minimum requirement #41135

Closed
wants to merge 1 commit into from
Closed

Switch to TLS 1.2 as minimum requirement #41135

wants to merge 1 commit into from

Conversation

loganaden
Copy link
Contributor

@loganaden loganaden commented Mar 21, 2020 •
edited by mit-mit
Loading

@kevmoo
Copy link
Member

kevmoo commented Mar 22, 2020

See https://dart-review.googlesource.com/c/sdk/+/140481

@kevmoo
Copy link
Member

kevmoo commented Mar 22, 2020

Thoughts @bkonyi ?

@bkonyi
Copy link
Contributor

bkonyi commented Mar 24, 2020

Thoughts @bkonyi ?

The change looks simple enough, but I'm not super familiar with that code so I'm going to loop in a couple of others as well on the CL.

@mit-mit mit-mit changed the title Switch to TLS 1.2 as minimum requirement according to https://tools.i… Switch to TLS 1.2 as minimum requirement Mar 30, 2020
@gustav3d
Copy link

Any progress here ?

@kevmoo
Copy link
Member

kevmoo commented Oct 17, 2020

CC @a-siva @vsmenon

@agilob
Copy link
Contributor

agilob commented Mar 25, 2021

TLS 1.0 and 1.1. are officially deprecated now https://datatracker.ietf.org/doc/rfc8996/ as of 2021-03-25

@aam aam mentioned this pull request Aug 4, 2021
Closed
@mit-mit
Copy link
Member

mit-mit commented Oct 4, 2021

Associated breaking change request: #46875

@mit-mit mit-mit mentioned this pull request Oct 4, 2021
Closed
@copybara-service copybara-service bot closed this in 8062c4c Oct 4, 2021
@mit-mit
Copy link
Member

mit-mit commented Oct 4, 2021

Thanks @loganaden for this contribution!

@EdgeKing810 EdgeKing810 deleted the tls branch November 13, 2021 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Maniakrzelaza Maniakrzelaza Maniakrzelaza approved these changes

Assignees
No one assigned
Labels
cla: yes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@loganaden @kevmoo @bkonyi @gustav3d @agilob @mit-mit @Maniakrzelaza @googlebot