Do not use sslname empty string for no ssl

diff --git a/modules/roles/manifests/historical_mirror.pp b/modules/roles/manifests/historical_mirror.pp
index d80b1af..5036e8f 100644 (file)
--- a/modules/roles/manifests/historical_mirror.pp
+++ b/modules/roles/manifests/historical_mirror.pp
@@ -29,7 +29,7 @@ class roles::historical_mirror {
                        tlsaport => [],
                }
        } else {
-               $sslname = ''
+               $sslname = undef
        }
 
        rsync::site_systemd { 'archive':

diff --git a/modules/rsync/manifests/site.pp b/modules/rsync/manifests/site.pp
index ba73c2c..04c1e20 100644 (file)
--- a/modules/rsync/manifests/site.pp
+++ b/modules/rsync/manifests/site.pp
@@ -5,7 +5,7 @@ define rsync::site (
        $content=undef,
        $max_clients=200,
        $ensure=present,
-       $sslname='',
+       $sslname=undef,
        $sslport=1873
 ){
 
@@ -62,7 +62,7 @@ define rsync::site (
                }
        }
 
-       if $sslname != '' {
+       if $sslname {
                file { "/etc/rsyncd-${name}-stunnel.conf":
                        content => template('rsync/rsyncd-stunnel.conf.erb'),
                        require => File["/etc/ssl/debian/certs/${sslname}.crt-chained"],