Skip to content

Avoid deprecated dsa for tests keys #820

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

Merged
merged 1 commit into from
Jul 5, 2025
Merged

Conversation

traylenator
Copy link
Contributor

Description

Running tests on a very modern system results in

ssh-keygen -t dsa
unknown key type dsa

with OpenSSH_9.9p1

@jcouball
Copy link
Member

jcouball commented Jul 4, 2025

Thank you, @traylenator! I appreciate the update to the test.

Can you please amend your commit message to be a conventional commit message? This will make sure this change shows up in our release notes via our automation. Something like:

test: avoid deprecated dsa for tests keys

```
ssh-keygen -t dsa
unknown key type dsa
```
with OpenSSH_9.9p1

@jcouball
Copy link
Member

jcouball commented Jul 4, 2025

If you'd rather amend your commit, let me know and I can make the change myself.

```
ssh-keygen -t dsa
unknown key type dsa
```
with OpenSSH_9.9p1
@jcouball
Copy link
Member

jcouball commented Jul 5, 2025

@traylenator

I figured out I was able to amend the commit myself and push the update.

Thank you for this contribution!

Copy link
Member

@jcouball jcouball left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯 This change is a welcome improvement! Thanks @traylenator.

@jcouball jcouball merged commit 1da8c28 into ruby-git:main Jul 5, 2025
7 checks passed
@traylenator
Copy link
Contributor Author

Thanks - am on a computer break now so would have been a while.

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

Successfully merging this pull request may close these issues.

2 participants