Micah Anderson [Sat, 19 Dec 2009 08:30:16 +0000 (03:30 -0500)]
fix the comments section so that the include isn't misleading. if you
use 'include sshd::debian', then none of the variables are set, and you
will fail to parse the templates
Micah Anderson [Thu, 9 Jul 2009 16:15:10 +0000 (12:15 -0400)]
fix previous change which took the client/server packages out of the linux class
and instead allow for a version change through an if variable. thanks ng!
Micah Anderson [Wed, 8 Jul 2009 01:02:31 +0000 (21:02 -0400)]
the sshd::linux class cannot also define the openssh package
if we are to have the possibility of potentially overriding the version number it must be done in the base class
Micah Anderson [Wed, 8 Jul 2009 00:55:13 +0000 (20:55 -0400)]
Template out the possibility of specifying the key word 'off' to the
$sshd_port parameter, which simply puts a comment in front of that
option, rather than specifying it.
Micah Anderson [Wed, 8 Jul 2009 00:55:01 +0000 (20:55 -0400)]
make it possible to override what version of openssh-server and client are installed by providing the variable $sshd_ensure_version, which defaults to the previous value of present when not specified
Micah Anderson [Wed, 8 Jul 2009 00:52:40 +0000 (20:52 -0400)]
replace the sshd_additional_options variable with two, one called
sshd_head_additional_options and one called sshd_tail_additional_options.
the first puts the value at the beginning of the file, and the second at
the end.
This is necessary due to some option ordering requiring things to be
before others
Micah Anderson [Sun, 7 Dec 2008 17:15:41 +0000 (12:15 -0500)]
Fix location of default sftp-server on Debian, and uncomment the sftp
configuration line to get the sshd_config file defaults to be more
like the standard shipped defaults from Debian
Micah Anderson [Mon, 27 Oct 2008 21:00:39 +0000 (17:00 -0400)]
In debian, the daemon is run as 'sshd', but the initscript is
/etc/init.d/ssh, which means that name needs to be set to 'ssh', and
pattern needs to be set to 'sshd', and then we set the hassstatus and
hasrestart depending on the lsbdistcodename
Micah Anderson [Mon, 27 Oct 2008 19:45:11 +0000 (15:45 -0400)]
Debian's sshd config typically has PrintMotd no set because its already printed via PAM.
Without it set, it is defaulted to 'yes', which results in the MOTD being printed twice, so
we return the Debian default configuration in this commit
Micah Anderson [Sun, 26 Oct 2008 16:39:45 +0000 (12:39 -0400)]
change the debian 'hasrestart' option to a selector based on which $debian_version is detected,
etch does not have a ssh restart option in the initscript, but lenny does
Micah Anderson [Thu, 23 Oct 2008 19:04:47 +0000 (15:04 -0400)]
update formatting to be consistent with upstream puppet emacs mode, if this is different from the vim
mode, then there is a difference between these two editor's formatting that needs to be resolved
Micah Anderson [Thu, 23 Oct 2008 18:59:42 +0000 (14:59 -0400)]
debian has both status and restart options, in fact restart is
preferable because a stop/start operation can leave sshd broken
because the stop wont stop before the start is run. On the next puppet
run ssh will be brought back up, but its a hair-raising few minutes
while you wonder what happened
Micah Anderson [Sat, 27 Sep 2008 21:45:57 +0000 (17:45 -0400)]
clarify in the example about how you can set multiple ListenAddresses with the right syntax
also set the default to be 0.0.0.0 and :: which is the normal default for all IPv4 and all IPv6 addresses
Micah Anderson [Sat, 27 Sep 2008 20:42:08 +0000 (16:42 -0400)]
Change the template naming:
1. remove the _normal suffix, as it is not used
2. add a selector to look for the variable $lsbdistcodename being set and use that in selecting a template
this is useful to create a Debian_Etch.erb and a Debian_Lenny.erb which can have different values. For example
the Debian Etch version of openssh does not have the AllowAgentForwarding option, and if it is included, ssh will
fail to start
Micah Anderson [Fri, 26 Sep 2008 21:55:02 +0000 (17:55 -0400)]
add sshd_allow_tcp_forwarding variable, with the default changed to no (note this is opposite of
what the existing template had enabled for Debian, but this is a better setting)
Micah Anderson [Fri, 26 Sep 2008 20:59:55 +0000 (16:59 -0400)]
add comment about PAM auth to the Debian template (copied from the Gentoo one), as its useful info to
have available, due to the complexity of the option
Micah Anderson [Fri, 26 Sep 2008 20:57:59 +0000 (16:57 -0400)]
Add the variable AllowAgentForwarding to be set, with the default of 'no', only the Debian
template was adjusted for this, as my knowledge of the other operating systems is not good enough
to determine the appropriate setting there