sous-chefs / firewall

Development repository for the firewall cookbook
https://supermarket.chef.io/cookbooks/firewall
Apache License 2.0
99 stars 150 forks source link

Allow Optional Comments #195

Closed pwelch closed 6 years ago

pwelch commented 6 years ago

Description

Allow iptable rule comments to be excluded with a flag. Sometimes it might be necessary to not have a comment for a given rule.

Issues Resolved

Check List

martinb3 commented 6 years ago

Hi @pwelch -- thanks for this! I think the only thing missing is a test for omitting comments. Doesn't have to be much, but I figure we should probably include something, just so we don't break it in the future.

Maybe mark one of the existing test cookbook rules as include_comment false, and update the test for it to be sure?

pwelch commented 6 years ago

@martinb3 added a test on 0e6b024