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

diff --git a/modules/roles/manifests/mailrelay.pp b/modules/roles/manifests/mailrelay.pp
index b17bb01..2fba6a6 100644 (file)
--- 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 b985cd0..c184f96 100644 (file)
--- 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,
+          }
         }
       }
     }