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

Update SSH key to RSA now that DSA no longer works #404

Merged
merged 1 commit into from Jun 23, 2016

Conversation

@metajack
Copy link
Contributor

metajack commented Jun 23, 2016

This change is Reviewable

@edunham
Copy link
Contributor

edunham commented Jun 23, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Jun 23, 2016

📌 Commit e874fc8 has been approved by edunham

@bors-servo
Copy link
Contributor

bors-servo commented Jun 23, 2016

Testing commit e874fc8 with merge e89bfa8...

bors-servo added a commit that referenced this pull request Jun 23, 2016
Update SSH key to RSA now that DSA no longer works

<!-- Reviewable:start -->
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/404)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jun 23, 2016

☀️ Test successful - travis

@bors-servo bors-servo merged commit e874fc8 into servo:master Jun 23, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@metajack metajack deleted the metajack:ssh-jack-rsa branch Jun 23, 2016
@aneeshusa
Copy link
Member

aneeshusa commented Jun 27, 2016

DSA keys are certainly weak, so I'm glad to see them go. However, while we are in the process of standardizing and securing our OpenSSH configuration, I personally would like to see everyone move to Ed25519 keys instead of RSA keys. Everyone will need to change how they connect anyways for #253, so I think this would be a nice opportunity to also bump our key strength and take advantage of ECC. Thoughts?

(Note that Ed25519 keys won't work just yet, they will need some Salt work.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.