diff options
| author | Christian Breunig <christian@breunig.cc> | 2024-04-01 21:46:13 +0200 |
|---|---|---|
| committer | Christian Breunig <christian@breunig.cc> | 2024-04-01 21:53:08 +0200 |
| commit | 6e0fdbcbba39691461f791c7a68a2c6c5091d2c1 (patch) | |
| tree | e51a1e7b29873191d4346c38eb128048757e5aac /python | |
| parent | ecdf22fee3272dedc8c1c7c6d5e95057042b48ce (diff) | |
| download | vyos-1x-6e0fdbcbba39691461f791c7a68a2c6c5091d2c1.tar.gz vyos-1x-6e0fdbcbba39691461f791c7a68a2c6c5091d2c1.zip | |
firewall: T2199: always use full nft command name (e.g. --file over -f)
Diffstat (limited to 'python')
| -rw-r--r-- | python/vyos/firewall.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/python/vyos/firewall.py b/python/vyos/firewall.py index e70b4f0d9..e29aeb0c6 100644 --- a/python/vyos/firewall.py +++ b/python/vyos/firewall.py @@ -66,7 +66,7 @@ def fqdn_config_parse(firewall): rule = path[4] suffix = path[5][0] set_name = f'{hook_name}_{priority}_{rule}_{suffix}' - + if (path[0] == 'ipv4') and (path[1] == 'forward' or path[1] == 'input' or path[1] == 'output' or path[1] == 'name'): firewall['ip_fqdn'][set_name] = domain elif (path[0] == 'ipv6') and (path[1] == 'forward' or path[1] == 'input' or path[1] == 'output' or path[1] == 'name'): @@ -85,7 +85,7 @@ def fqdn_resolve(fqdn, ipv6=False): def find_nftables_rule(table, chain, rule_matches=[]): # Find rule in table/chain that matches all criteria and return the handle - results = cmd(f'sudo nft -a list chain {table} {chain}').split("\n") + results = cmd(f'sudo nft --handle list chain {table} {chain}').split("\n") for line in results: if all(rule_match in line for rule_match in rule_matches): handle_search = re.search('handle (\d+)', line) @@ -655,7 +655,7 @@ def geoip_update(firewall, force=False): 'ipv6_sets': ipv6_sets }) - result = run(f'nft -f {nftables_geoip_conf}') + result = run(f'nft --file {nftables_geoip_conf}') if result != 0: print('Error: GeoIP failed to update firewall') return False |