diff options
| author | Christian Breunig <christian@breunig.cc> | 2023-11-11 13:05:42 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-11-11 13:05:42 +0100 |
| commit | e5a53d48cf149facb30410c25e55ab3780205186 (patch) | |
| tree | 0ea5b052b18f9db57fc65981950529ef478e8298 /python/vyos | |
| parent | 2ca0ac6ac420b5904d87344db80840c640d2cde9 (diff) | |
| parent | c4409d6a4e11bf2acc7b5b96888e2c471c4559e5 (diff) | |
| download | vyos-1x-e5a53d48cf149facb30410c25e55ab3780205186.tar.gz vyos-1x-e5a53d48cf149facb30410c25e55ab3780205186.zip | |
Merge pull request #2471 from nicolas-fort/T5729
T5729: firewall: switch to valueless in
Diffstat (limited to 'python/vyos')
| -rw-r--r-- | python/vyos/firewall.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/python/vyos/firewall.py b/python/vyos/firewall.py index dc5787595..a2622fa00 100644 --- a/python/vyos/firewall.py +++ b/python/vyos/firewall.py @@ -87,7 +87,6 @@ def nft_action(vyos_action): def parse_rule(rule_conf, hook, fw_name, rule_id, ip_name): output = [] - #def_suffix = '6' if ip_name == 'ip6' else '' if ip_name == 'ip6': def_suffix = '6' @@ -97,7 +96,7 @@ def parse_rule(rule_conf, hook, fw_name, rule_id, ip_name): family = 'bri' if ip_name == 'bri' else 'ipv4' if 'state' in rule_conf and rule_conf['state']: - states = ",".join([s for s, v in rule_conf['state'].items() if v == 'enable']) + states = ",".join([s for s in rule_conf['state']]) if states: output.append(f'ct state {{{states}}}') @@ -395,7 +394,7 @@ def parse_rule(rule_conf, hook, fw_name, rule_id, ip_name): if 'priority' in rule_conf['vlan']: output.append(f'vlan pcp {rule_conf["vlan"]["priority"]}') - if 'log' in rule_conf and rule_conf['log'] == 'enable': + if 'log' in rule_conf: action = rule_conf['action'] if 'action' in rule_conf else 'accept' #output.append(f'log prefix "[{fw_name[:19]}-{rule_id}-{action[:1].upper()}]"') output.append(f'log prefix "[{family}-{hook}-{fw_name}-{rule_id}-{action[:1].upper()}]"') |