From: Peter Palfrader <peter@palfrader.org>
Date: Sat, 21 Sep 2019 22:23:58 +0000 (+0200)
Subject: Set port to 25 explicitly instead of undef if we do not have it overwritten for this... 
X-Git-Url: https://git.adam-barratt.org.uk/?a=commitdiff_plain;h=623a535ff7b70ac98e56551378eacf39da65e0cf;p=mirror%2Fdsa-puppet.git

Set port to 25 explicitly instead of undef if we do not have it overwritten for this host
---

diff --git a/modules/roles/manifests/mailrelay.pp b/modules/roles/manifests/mailrelay.pp
index b17bb019d..2fba6a6aa 100644
--- a/modules/roles/manifests/mailrelay.pp
+++ b/modules/roles/manifests/mailrelay.pp
@@ -28,7 +28,7 @@ class roles::mailrelay {
   @@ferm::rule::simple { "dsa-smtp-from-mailrelay-${::fqdn}":
     tag         => 'smtp::server::from::mailrelay',
     description => 'Allow smtp access from a mailrelay',
-    port        => '25',
+    port        => '7', # will be overwritten on collection
     saddr       => $base::public_addresses,
   }
 }
diff --git a/modules/roles/manifests/mta.pp b/modules/roles/manifests/mta.pp
index b985cd03a..c184f968d 100644
--- a/modules/roles/manifests/mta.pp
+++ b/modules/roles/manifests/mta.pp
@@ -33,7 +33,10 @@ class roles::mta(
         }
 
         Ferm::Rule::Simple <<| tag == 'smtp::server::from::mailrelay' |>> {
-          port => $mailport
+          port => $mailport == undef ? {
+            true    => 25,
+            default => $mailport,
+          }
         }
       }
     }