From 19c71d9abd269d88450f54ddb95e036c804a8fe8 Mon Sep 17 00:00:00 2001 From: Thore Bödecker Date: Tue, 24 Sep 2019 16:35:19 +0200 Subject: disable conntrack filtering in FORWARD/OUTPUT conntrack filtering basically doesn't work in those chains, so we need to disable it. --- manifests/config.pp | 4 ++-- spec/acceptance/ferm_spec.rb | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/manifests/config.pp b/manifests/config.pp index 7dae7a5..16ecd9e 100644 --- a/manifests/config.pp +++ b/manifests/config.pp @@ -49,12 +49,12 @@ class ferm::config { } ferm::chain{'FORWARD': policy => $ferm::forward_policy, - disable_conntrack => $ferm::disable_conntrack, + disable_conntrack => true, log_dropped_packets => $ferm::forward_log_dropped_packets, } ferm::chain{'OUTPUT': policy => $ferm::output_policy, - disable_conntrack => $ferm::disable_conntrack, + disable_conntrack => true, log_dropped_packets => $ferm::output_log_dropped_packets, } diff --git a/spec/acceptance/ferm_spec.rb b/spec/acceptance/ferm_spec.rb index c5018da..f827dc2 100644 --- a/spec/acceptance/ferm_spec.rb +++ b/spec/acceptance/ferm_spec.rb @@ -32,7 +32,7 @@ basic_manifest = %( manage_configfile => true, manage_initfile => #{manage_initfile}, # CentOS-6 does not provide init script forward_policy => 'DROP', - output_policy => 'DROP', + output_policy => 'ACCEPT', input_policy => 'DROP', rules => { 'allow_acceptance_tests' => { @@ -66,7 +66,7 @@ describe 'ferm' do end describe command('iptables-save') do - its(:stdout) { is_expected.to match %r{.*filter.*:INPUT DROP.*:FORWARD DROP.*:OUTPUT DROP.*}m } + its(:stdout) { is_expected.to match %r{.*filter.*:INPUT DROP.*:FORWARD DROP.*:OUTPUT ACCEPT.*}m } end describe iptables do -- cgit v1.2.3