From: Tom Hughes <tom@compton.nu>
Date: Tue, 10 Feb 2015 10:16:15 +0000 (+0000)
Subject: Don't use only_if on firewall_rule as it's a definition
X-Git-Url: https://git.openstreetmap.org/chef.git/commitdiff_plain/a8942b825af38bee6da538a5a64dcce91c42341a?ds=sidebyside

Don't use only_if on firewall_rule as it's a definition
---

diff --git a/cookbooks/exim/recipes/default.rb b/cookbooks/exim/recipes/default.rb
index 099e047c6..1bbd084ee 100644
--- a/cookbooks/exim/recipes/default.rb
+++ b/cookbooks/exim/recipes/default.rb
@@ -144,11 +144,12 @@ else
   end
 end
 
-firewall_rule "deny-outbound-smtp" do
-  action :reject
-  source "fw"
-  dest "net"
-  proto "tcp:syn"
-  dest_ports "smtp"
-  only_if { node[:exim][:smarthost_via] }
+if node[:exim][:smarthost_via] # ~FC023
+  firewall_rule "deny-outbound-smtp" do
+    action :reject
+    source "fw"
+    dest "net"
+    proto "tcp:syn"
+    dest_ports "smtp"
+  end
 end