diff --git a/README.md b/README.md index 8361fe3..f88c2b6 100644 --- a/README.md +++ b/README.md @@ -25,11 +25,13 @@ ## Features - install required binaries -- manage local custom configuration based on parameters, overriding the defaults - manage selinux rules - manage service - (optional) manage firewall - + +> Note +Originally I thought it was a great idea to have a single custom configuration file, which would override the default settings in the sshd_config. But I quickly learned, that this is not a great idea, for it seemed to work out differently on various hosts despite identical settings. For instance gitlab is using SSH not only for remote sessions but also for git operations, which broke with this file in place. So right now this module will only ensure the main default config is available and the service is always running. Custom configurations can be enabled after through testing through the parameters under #sshd section, but can be forced off through the `ssh_custom_ensure`setting set to `absent` (default). + ## Support - Rocky 9 (Any RHEL 9 based OS should work but has not been tested) diff --git a/doc/file.README.html b/doc/file.README.html index 92c2f45..d50ddf8 100644 --- a/doc/file.README.html +++ b/doc/file.README.html @@ -97,8 +97,6 @@
install required binaries
manage local custom configuration based on parameters, overriding the defaults
-manage selinux rules
manage service
@@ -106,6 +104,10 @@(optional) manage firewall
++Note Originally I thought it was a great idea to have a single custom configuration file, which would override the default settings in the sshd_config. But I quickly learned, that this is not a great idea, for it seemed to work out differently on various hosts despite identical settings. For instance gitlab is using SSH not only for remote sessions but also for git operations, which broke with this file in place. So right now this module will only ensure the main default config is available and the service is always running. Custom configurations can be enabled after through testing through the parameters under #sshd section, but can be forced off through the
+ssh_custom_ensuresetting set toabsent(default).
Rocky 9 (Any RHEL 9 based OS should work but has not been tested)
diff --git a/doc/index.html b/doc/index.html index f4a1c4f..b5751d3 100644 --- a/doc/index.html +++ b/doc/index.html @@ -97,8 +97,6 @@install required binaries
manage local custom configuration based on parameters, overriding the defaults
-manage selinux rules
manage service
@@ -106,6 +104,10 @@(optional) manage firewall
++Note Originally I thought it was a great idea to have a single custom configuration file, which would override the default settings in the sshd_config. But I quickly learned, that this is not a great idea, for it seemed to work out differently on various hosts despite identical settings. For instance gitlab is using SSH not only for remote sessions but also for git operations, which broke with this file in place. So right now this module will only ensure the main default config is available and the service is always running. Custom configurations can be enabled after through testing through the parameters under #sshd section, but can be forced off through the
+ssh_custom_ensuresetting set toabsent(default).
Rocky 9 (Any RHEL 9 based OS should work but has not been tested)
diff --git a/doc/puppet_classes/confdroid_ssh_3A_3Amain_3A_3Afiles.html b/doc/puppet_classes/confdroid_ssh_3A_3Amain_3A_3Afiles.html index 008c095..6ffa772 100644 --- a/doc/puppet_classes/confdroid_ssh_3A_3Amain_3A_3Afiles.html +++ b/doc/puppet_classes/confdroid_ssh_3A_3Amain_3A_3Afiles.html @@ -134,14 +134,7 @@ 36 37 38 -39 -40 -41 -42 -43 -44 -45 -46 +39# File 'manifests/main/files.pp', line 6
@@ -178,13 +171,6 @@ class confdroid_ssh::main::files (
content => template($sshd_custom_erb),
notify => Service[$sshd_service],
}
- # we want the default root login setting to be managed by the custom conf,
- # so we remove the default file if it exists
- file { $sshd_root_login_file:
- ensure => absent,
- path => $sshd_root_login_file,
- notify => Service[$sshd_service],
- }
}
}