From 267f5e7f27ff8b0cfaca8cd6ff6a30cc0fef5609 Mon Sep 17 00:00:00 2001 From: Daniel Black Date: Sun, 18 Dec 2011 20:16:19 +1100 Subject: [PATCH] (#12897) Require jump=>LOG for log params This validates that when log_prefix or log_level is specified the jump should be 'LOG' --- lib/puppet/type/firewall.rb | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/lib/puppet/type/firewall.rb b/lib/puppet/type/firewall.rb index f8cc358..4843895 100644 --- a/lib/puppet/type/firewall.rb +++ b/lib/puppet/type/firewall.rb @@ -570,6 +570,12 @@ Puppet::Type.newtype(:firewall) do end end + if value(:log_prefix) || value(:log_level) + unless value(:jump).to_s == "LOG" + self.fail "Parameter log_prefix and log_level require jump => LOG" + end + end + if value(:burst) && ! value(:limit) self.fail "burst makes no sense without limit" end -- 2.45.2