From 32c97619505a28d6f9838d9ebc109f472687fe4f Mon Sep 17 00:00:00 2001
From: Marknl <mark@e-rave.nl>
Date: Tue, 10 Jan 2017 11:21:20 +0100
Subject: Added $host option

Current rule.pp converts the $name into the "HOST" parameter in the file, this can result in these definitions:

```shorewall::host { 'eth0:$VPN_HOSTS':
  zone => 'vpn',
  options => 'ipsec',
  order => 200;
}```

I suggest moving the variable usage from the $name into a $host parameter, so above example becomes:

```shorewall::host { 'vpn-hosts':
  zone => 'vpn',
  host => 'eth0:$VPN_HOSTS',
  options => 'ipsec',
  order => 200;
}```
---
 manifests/host.pp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/manifests/host.pp b/manifests/host.pp
index f400223..2510ca7 100644
--- a/manifests/host.pp
+++ b/manifests/host.pp
@@ -1,10 +1,11 @@
 define shorewall::host(
     $zone,
+    $host,
     $options = 'tcpflags,blacklist,norfc1918',
     $order='100'
 ){
     shorewall::entry{"hosts-${order}-${name}":
-        line => "${zone} ${name} ${options}"
+        line => "#${name}\n${zone} ${host} ${options}"
     }
 }
 
-- 
cgit v1.2.3