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

servo should negotiate more modern secure ciphersuites #8594

Closed
alex opened this issue Nov 18, 2015 · 0 comments
Closed

servo should negotiate more modern secure ciphersuites #8594

alex opened this issue Nov 18, 2015 · 0 comments

Comments

@alex
Copy link
Contributor

alex commented Nov 18, 2015

Right now servo appears to use OpenSSL's defaults, which are very bad. It should use a modern ciphersuite string such as https://github.com/python/cpython/blob/master/Lib/ssl.py#L168-L172

You can see the list it's sending at https://howsmyssl.com

@jdm jdm added the A-network label Nov 19, 2015
bors-servo pushed a commit that referenced this issue Nov 20, 2015
Fixed #8594 -- use a more modern default cipher suite list by default

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8611)
<!-- Reviewable:end -->
bors-servo pushed a commit that referenced this issue Nov 20, 2015
Fixed #8594 -- use a more modern default cipher suite list by default

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8611)
<!-- Reviewable:end -->
bors-servo pushed a commit that referenced this issue Nov 20, 2015
Fixed #8594 -- use a more modern default cipher suite list by default

Fixes #8594.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8611)
<!-- Reviewable:end -->
bors-servo pushed a commit that referenced this issue Nov 20, 2015
Fixed #8594 -- use a more modern default cipher suite list by default

Fixes #8594.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8611)
<!-- Reviewable:end -->
bors-servo pushed a commit that referenced this issue Nov 20, 2015
Fixed #8594 -- use a more modern default cipher suite list by default

Fixes #8594.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8611)
<!-- Reviewable:end -->
bors-servo pushed a commit that referenced this issue Nov 20, 2015
Fixed #8594 -- use a more modern default cipher suite list by default

Fixes #8594.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8611)
<!-- Reviewable:end -->
bors-servo pushed a commit that referenced this issue Nov 21, 2015
Fixed #8594 -- use a more modern default cipher suite list by default

Fixes #8594.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8611)
<!-- Reviewable:end -->
bors-servo pushed a commit that referenced this issue Nov 21, 2015
Fixed #8594 -- use a more modern default cipher suite list by default

Fixes #8594.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8611)
<!-- Reviewable:end -->
bors-servo pushed a commit that referenced this issue Nov 21, 2015
Fixed #8594 -- use a more modern default cipher suite list by default

Fixes #8594.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8611)
<!-- Reviewable:end -->
bors-servo pushed a commit that referenced this issue Nov 21, 2015
Fixed #8594 -- use a more modern default cipher suite list by default

Fixes #8594.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8611)
<!-- Reviewable:end -->
paulrouget pushed a commit to paulrouget/servo that referenced this issue Nov 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants