Merge pull request #11 from bastelfreak/optional

dont make `$destination_addresses` as optional, remove useless validation
voxpupuli · Jul 27, 2021 · ac252b8 · ac252b8
2 parents 1495626 + 0c63cd6
commit ac252b8
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/manifests/interface.pp b/manifests/interface.pp
@@ -33,7 +33,7 @@
 define wireguard::interface (
   String[1] $public_key,
   String[1] $endpoint,
-  Optional[Array[Stdlib::IP::Address]] $destination_addresses = [$facts['networking']['ip'], $facts['networking']['ip6'],],
+  Array[Stdlib::IP::Address] $destination_addresses = [$facts['networking']['ip'], $facts['networking']['ip6'],],
   String[1] $interface = $title,
   Integer[1024, 65000] $dport = Integer(regsubst($title, '^\D+(\d+)$', '\1')),
   String[1] $input_interface = $facts['networking']['primary'],
@@ -44,8 +44,6 @@
   require wireguard
 
   if $manage_firewall {
-    assert_type(Array[Stdlib::IP::Address], $source_addresses)
-    assert_type(Array[Stdlib::IP::Address], $destination_addresses)
     ferm::rule { "allow_wg_${interface}":
       action    => 'ACCEPT',
       chain     => 'INPUT',