chef-cookbooks / iptables

Development repository for Chef Cookbook iptables

https://supermarket.chef.io/cookbooks/iptables

Apache License 2.0

102 stars 141 forks source link

Fix a nil safety issue to allow the cookbook to load #106

Closed jjustice6 closed 4 years ago

jjustice6 commented 4 years ago

Fix a nil safety issue to allow the cookbook to load at loadtime on unsupported platforms if it's in the dependency tree

Signed-off-by: Joshua Justice jjustice6@bloomberg.net

Description

This works around the problem by making the default property valid (an empty string), instead of nil.

Issues Resolved

105

Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[X] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

tas50 commented 4 years ago

Fixed via https://github.com/chef-cookbooks/iptables/commit/4a536f241f14b930ba4cd3610717f2aeaabe0a49