]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commit
Code Review / puppet-modules / puppetlabs-firewall.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 9ef77c2) | patch
Add sanity check for rule to hash parser
authorAndjelko Horvat <andjelko.horvat.comel@gmail.com>
Thu, 8 Dec 2016 22:28:04 +0000 (23:28 +0100)
committerAndjelko Horvat <andjelko.horvat.comel@gmail.com>
Thu, 8 Dec 2016 22:41:54 +0000 (23:41 +0100)
commit06cf2eeed850f3a04732ab53048e5b1f8d0956b8
tree70aadb64abb76ef4ca63c23dd3392942ab8188f8tree | snapshot
parent9ef77c2e765dfc18869659f76d3c984c0525c670commit | diff
Add sanity check for rule to hash parser

Parser fails in some cases with combined arguments e.g. if some other argument
is set between match name and match option, like dport in this example:

-m tcp --dport 80 --tcp-flags FIN,SYN,RST,ACK SYN

If the parsed keys and values count is not the same, the error is raised with
the problematic rule line.
lib/puppet/provider/firewall/iptables.rb diff | blob | history