Remove trash from the middle of the main parse

The regular parse mechanisms can handle table, chain, and comment(s) on
their own already, if a single space character is added to the head of
the rule.  :table was already in @resource_map, :chain is added to the
parse output by the unnecessary special mechanism anyway, and :name
already grabs ALL comments successfully since the escaping fixes.
'line' is used instead of 'line.dup' because string addition returns a
new string, so String#dup would be redundant.

Author: greatflyingsteve

lib/puppet/provider/firewall/ip6tables.rb

@@ -93,6 +93,7 @@ def self.iptables_save(*args)
 
   @resource_map = {
     burst: '--limit-burst',
+    chain: '-A',
     checksum_fill: '--checksum-fill',
     clamp_mss_to_pmtu: '--clamp-mss-to-pmtu',
     condition: '--condition',

lib/puppet/provider/firewall/iptables.rb

@@ -85,6 +85,7 @@
 
   @resource_map = {
     burst: '--limit-burst',
+    chain: '-A',
     checksum_fill: '--checksum-fill',
     clamp_mss_to_pmtu: '--clamp-mss-to-pmtu',
     condition: '--condition',
@@ -512,7 +513,7 @@ def self.instances
   def self.rule_to_hash(line, table, counter)
     hash = {}
     keys = []
-    values = line.dup
+    values = ' ' + line
 
     ####################
     # PRE-PARSE CLUDGING
@@ -656,23 +657,6 @@ def self.rule_to_hash(line, table, counter)
       end
     end
 
-    # Manually remove chain
-    if %r{(\s|^)-A\s}.match?(values)
-      values = values.sub(%r{(\s|^)-A\s}, '\1')
-      keys << :chain
-    end
-
-    # Manually remove table (used in some tests)
-    if %r{^-t\s}.match?(values)
-      values = values.sub(%r{^-t\s}, '')
-      keys << :table
-    end
-
-    # manually remove comments if they made it this far
-    if %r{-m comment --comment}.match?(values)
-      values = values.sub(%r{-m comment --comment "((?:\\"|[^"])*)"}, {})
-    end
-
     valrev = values.scan(%r{("([^"\\]|\\.)*"|\S+)}).transpose[0].reverse
 
     if keys.length != valrev.length