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

Introducing find_ssh_version and guess_ssh_version #132

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JHeinzde
Copy link

@JHeinzde JHeinzde commented Jun 20, 2019

This PR contains an implementation for find_ssh_version and guess_ssh_version. These provide the same behaviors as the methods in devsec_ssh.rb in the chef ssh hardening cookbook. This PR is the first in a series of PRs that will try to add the behavior of the chef ssh hardening implementation. Related to #126

…ified. In that case return a fallback with guess_ssh_version. This behavior is identical to devsec_ssh.rb in the chef ssh baseline. Also adding a find_ssh_version method that behaves almost exactly like the one in devsec_ssh.rb.

Signed-off-by: jheinz <[email protected]>
@JHeinzde JHeinzde force-pushed the refactoring/ssh_crypto branch from 5b8e2bf to ee10de4 Compare June 20, 2019 13:13
@chris-rock
Copy link
Member

@JHeinzde I am sorry I missed this PR. This looks like an awesome improvement. I am going to test it within the next week. Can you make sure that Travis will be happy?

@artem-sidorenko
Copy link
Member

@chris-rock don't wait for Jonathan, he won't be able to answer anything in the next weeks or maybe even several months (due to accident) :-(

Feel free to proceed with review - I'll try to help/takeover if/when I've time

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.

3 participants