This repository has been archived by the owner on Nov 8, 2021. It is now read-only.
Ubuntu #83
Merged
Ubuntu #83
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sshd's AuthorizedKeysCommand setting install did not work on Debian-based distributions, since these directives: - AuthorizedKeysCommand - AuthorizedKeysCommandUser are not present by default in sshd_config and the sed substitutions did not work. New checks are implemented in order to substitute or add corrected lines of sshd configuration.
* Fix sshd AuthorizedKeysCommand setting problem sshd's AuthorizedKeysCommand setting install did not work on Debian-based distributions, since these directives: - AuthorizedKeysCommand - AuthorizedKeysCommandUser are not present by default in sshd_config and the sed substitutions did not work. New checks are implemented in order to substitute or add corrected lines of sshd configuration. * Manage SSHd service name Switch between ssh and sshd service name, based on Linux distribution base. * Bugfix on a case of sshd config content Fix problem when AuthorizedKeysCommandUser is commented and it was not replaced.
|
This change breaks functionality on CENTOS7 because it's checking for init.d scripts and doesn't take into account systemd, the restart method should be systemctl restart sshd. This causes script to throw an error since ssh.service is not found, and would break in any cloudformation scripts. |
Closed
|
hi @nicholascowan centos is not one of the supported distros at the moment. I opened an issue #84 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fixes #81
fixes #72