shorten up the typing

Signed-off-by: Stephen Gran <steve@lobefin.net>

diff --git a/modules/apache2/manifests/init.pp b/modules/apache2/manifests/init.pp
index cd2a4e40add703b7b7b5c0957fe19c558970357e..6235ed3a212e7e3edb621e667b12ca7e0d8b3b1c 100644 (file)
--- a/modules/apache2/manifests/init.pp
+++ b/modules/apache2/manifests/init.pp
@@ -132,6 +132,6 @@ class apache2 {
         @ferm::rule { "dsa-apache":
                 domain          => "(ip ip6)",
                 description     => "Allow web access",
-                rule            => "proto tcp mod state state (NEW) dport (80) ACCEPT"
+                rule            => "&SERVICE(tcp, 80)"
         }
 }

diff --git a/modules/exim/manifests/init.pp b/modules/exim/manifests/init.pp
index 858527909dc16ff7f79be6a33b13cbaf563eadb1..6856da2c87b559b2b3aa2e9d5f04f4b473fb270c 100644 (file)
--- a/modules/exim/manifests/init.pp
+++ b/modules/exim/manifests/init.pp
@@ -159,6 +159,6 @@ class exim {
     @ferm::rule { "dsa-exim":
             domain          => "(ip ip6)",
             description     => "Allow smtp access",
-            rule            => "proto tcp mod state state (NEW) dport (25) ACCEPT"
+            rule            => "&SERVICE(tcp, 25)"
     }
 }

diff --git a/modules/ferm/files/defs.conf b/modules/ferm/files/defs.conf
index 0359fa921814bb42101ef0ec2f27601506e8b58e..b78b9abf95183685073ecb07adf90397611d54a6 100644 (file)
--- a/modules/ferm/files/defs.conf
+++ b/modules/ferm/files/defs.conf
@@ -4,23 +4,16 @@
 ##
 
 @def &SERVICE($proto, $port) = {
- domain (ip ip6) chain INPUT proto $proto dport $port ACCEPT;
+ proto $proto mod state state (NEW) dport $port ACCEPT;
 }
 
-@def &V4_SERVICE($proto, $port) = {
- domain ip chain INPUT proto $proto dport $port ACCEPT;
+@def &SERVICE_RANGE($proto, $port, $srange) = {
+ proto $proto mod state state (NEW) dport $port saddr ($srange) ACCEPT;
 }
 
-@def &V6_SERVICE($proto, $port) = {
- domain ip6 chain INPUT proto $proto dport $port ACCEPT;
-}
-
-@def &V4_SERVICE_RANGE($proto, $port, $srange) = {
- domain ip chain INPUT proto $proto dport $port saddr $srange ACCEPT;
-}
-
-@def &V6_SERVICE_RANGE($proto, $port, $srange) = {
- domain ip6 chain INPUT proto $proto dport $port saddr $srange ACCEPT;
+@def &TCP_UDP_SERVICE($port) = {
+ proto tcp mod state state (NEW) dport $port ACCEPT;
+ proto udp mod state state (NEW) dport $port ACCEPT;
 }
 
 @def $HOST_MUNIN  = (192.25.206.57 192.25.206.33);

diff --git a/modules/named/manifests/init.pp b/modules/named/manifests/init.pp
index 65d4cc5f1ab40a0ae0a23be819bf99cac10d7196..719c0e7b4b18bdc4401a1df397b15d6480135256 100644 (file)
--- a/modules/named/manifests/init.pp
+++ b/modules/named/manifests/init.pp
@@ -28,7 +28,7 @@ class named {
         @ferm::rule { "dsa-bind":
                 domain          => "(ip ip6)",
                 description     => "Allow nameserver access",
-                rule            => "proto (udp tcp) mod state state (NEW) dport (53) ACCEPT"
+                rule            => "&TCP_UDP_SERVICE(53)"
         }
 }
 

diff --git a/modules/ntp/manifests/init.pp b/modules/ntp/manifests/init.pp
index ace2f8f8dbb886e257aa06668bf33a54b655c124..af086f6849edc5621f6c8d2fc621e3c7357c8af5 100644 (file)
--- a/modules/ntp/manifests/init.pp
+++ b/modules/ntp/manifests/init.pp
@@ -28,6 +28,6 @@ class ntp {
         @ferm::rule { "dsa-ntp":
                 domain          => "(ip ip6)",
                 description     => "Allow ntp access",
-                rule            => "proto udp mod state state (NEW) dport (123) ACCEPT"
+                rule            => "&SERVICE(udp, 123)"
         }
 }