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 099e047..1bbd084 100644 (file)
--- 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