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

included user input for missing vars, disabling passwordAuth and enab… #9

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

Conversation

17Halbe
Copy link

@17Halbe 17Halbe commented Apr 7, 2022

Hi there,
I changed the script to my liking and thought you might be interested. I removed the necessity to use commandline options and inserted user input demands for the not provided (and neccessary) options.
Additionally added the option to modify the sshd_config file on the host to disallow passwordAuthentication and allow PubkeyAuthentication if desired. It also checks if those config changes would be able to be made due to permission restrictions.
It furthermore tries to get the currently used sshd_config file (ps aux | grep ...)
Some documentation changed and the information about a daemon reload was added additionally with a final login check.

Hope that was all and understandable. ;) Thanks for your work and sorry for the hasty PR, I'm in a hurry!

@centic9
Copy link
Owner

centic9 commented Apr 7, 2022

Thanks for the proposed changes. I will merge at least some of it when I find some time to take a closer look!

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