-
Notifications
You must be signed in to change notification settings - Fork 76
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
RHEL6 accepts crypto settings from openssh 5.9 #129
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wojciech-kopras this looks great. Thank you. @artem-sidorenko do you think this is good to go?
@wojciech-kopras can you make the DCO bot happy? |
Signed-off-by: Wojciech Kopras <[email protected]>
Looks good to me, I'll cross-check it with chef-ssh-hardening, I also want to verify if this support was backported by RH (because of version detection logic in chef-ssh-hardening). Maybe we have here some edge case, which wasn't covered in the past, not sure yet |
@artem-sidorenko Once you're okay, we can merge this this PR. |
I'm currently on vacation, will have a look by the end of next week.
…--
Artem Sidorenko
Sent from my mobile
On 22. Aug 2019, at 23:49, Christoph Hartmann ***@***.***> wrote:
receiving
|
I'm not really sure how to handle this.. Sorry if it takes more time again, but I'll have to find another time to technically cross-check this with ansible-ssh-hardening and chef-ssh-hardening in order to know what would break and how it can be handled. |
@artem-sidorenko Can we just activate this on the latest 6.x version of Redhat/Centos? |
To fix #125