puppetlabs/puppetlabs-firewall — pull request #21 by kbarber (2011-10-26 01:22:52+00:00)
Title: (#10002) Change to dport and sport to handle ranges, and fix handling of 

(#10002) Change to dport and sport to handle ranges, and fix handling of name to port.

We hadn't been allowing ranges of the kind 22:1000 for ranges. This patch
fixes that. Thanks to Jason Hancock for finding this issue and providing a sample
patch.

Instead of using colon though, it was decided we would use a hyphen to specify a range
as its more agnostic. This patch does the filtering for both writing the rule and
reading the rule.

Also - the way we were doing name to port conversion had been broken. I found
this out while fixing the ranges, and have now fixed it and added tests.