Merge branch 'jenkins-build-13' into 'master'

Auto-merge for build 13

See merge request puppet/confdroid_ssh!13

doc/puppet_classes/confdroid_ssh_3A_3Afirewall_3A_3Aiptables.html

@@ -120,7 +120,9 @@
 22
 23
 24
-25</pre>
+25
+26
+27</pre>
       </td>
       <td>
         <pre class="code"><span class="info file"># File 'manifests/firewall/iptables.pp', line 6</span>
@@ -128,22 +130,24 @@
 class confdroid_ssh::firewall::iptables (
 
 ) inherits confdroid_ssh::params {
-  case $ssh_use_firewall {
+  if $ssh_use_firewall == true {
-    true, &#39;true&#39;, &#39;yes&#39;, &#39;1&#39;: {
-      $ssh_fw_ensure = &#39;present&#39;
-    }
-    default: {
-      $ssh_fw_ensure = &#39;absent&#39;
-    }
-  }
-
     firewall { &quot;${ssh_fw_order}${ssh_fw_port} allow SSH on port ${ssh_fw_port}&quot;:
-    ensure =&gt; $ssh_fw_ensure,
+      ensure =&gt; &#39;present&#39;,
       proto  =&gt; &#39;tcp&#39;,
       source =&gt; $ssh_source_range,
       dport  =&gt; $ssh_fw_port,
       jump   =&gt; &#39;accept&#39;,
     }
+  }
+  if $ssh_use_firewall == false {
+    firewall { &quot;${ssh_fw_order}${ssh_fw_port} remove SSH on port ${ssh_fw_port}&quot;:
+      ensure =&gt; &#39;absent&#39;,
+      proto  =&gt; &#39;tcp&#39;,
+      source =&gt; $ssh_source_range,
+      dport  =&gt; $ssh_fw_port,
+      jump   =&gt; &#39;accept&#39;,
+    }
+  }
 }</pre>
       </td>
     </tr>

manifests/firewall/iptables.pp

@@ -6,20 +6,22 @@
 class confdroid_ssh::firewall::iptables (
 
 ) inherits confdroid_ssh::params {
-  case $ssh_use_firewall {
+  if $ssh_use_firewall == true {
-    true, 'true', 'yes', '1': {
-      $ssh_fw_ensure = 'present'
-    }
-    default: {
-      $ssh_fw_ensure = 'absent'
-    }
-  }
-
     firewall { "${ssh_fw_order}${ssh_fw_port} allow SSH on port ${ssh_fw_port}":
-    ensure => $ssh_fw_ensure,
+      ensure => 'present',
       proto  => 'tcp',
       source => $ssh_source_range,
       dport  => $ssh_fw_port,
       jump   => 'accept',
     }
   }
+  if $ssh_use_firewall == false {
+    firewall { "${ssh_fw_order}${ssh_fw_port} remove SSH on port ${ssh_fw_port}":
+      ensure => 'absent',
+      proto  => 'tcp',
+      source => $ssh_source_range,
+      dport  => $ssh_fw_port,
+      jump   => 'accept',
+    }
+  }
+}