36 changed files with 213 additions and 1956 deletions
--- a/.ansible-lint
+++ b/.ansible-lint
@ -1,4 +0,0 @@
 skip_list:
  - command-instead-of-module
  - no-changed-when
  - role-name
--- a/.github/workflows/main.yml
+++ b/.github/workflows/main.yml
@ -1,23 +0,0 @@
 ---
 name: ipr-cnrs.nftables.molecule
 on:
  push:
    branches: [master]
  pull_request:
    branches: [master]
  workflow_dispatch:
 jobs:
  test:
    runs-on:  ubuntu-latest
    steps:
      - name: checkout
        uses: actions/checkout@v2
        with:
          path: "${{ github.repository }}"
      - name: molecule
        uses: robertdebock/molecule-action@2.6.17
--- a/.yamllint
+++ b/.yamllint
@ -1,33 +0,0 @@
 ---
 # Based on ansible-lint config
 extends: default
 rules:
  braces:
    max-spaces-inside: 1
    level: error
  brackets:
    max-spaces-inside: 1
    level: error
  colons:
    max-spaces-after: -1
    level: error
  commas:
    max-spaces-after: -1
    level: error
  comments: disable
  comments-indentation: disable
  document-start: disable
  empty-lines:
    max: 3
    level: error
  hyphens:
    level: error
  indentation: disable
  key-duplicates: enable
  line-length: disable
  new-line-at-end-of-file: disable
  new-lines:
    type: unix
  trailing-spaces: disable
  truthy: disable
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@ -1,99 +1,11 @@
 ## v2.0.1
 ### Added
 * Molecule tests for Gentoo (many thanks to @VTimofeenko ! − PR #25).
 ### Fixed
 * Separate repositories update from installation task (fix #24).
 ## v2.0.0
 ### Added
 * New examples usecases (mostly for playbooks) in README.md.
 * New rules (disable by default) can be define in *forward* chain (thanks to
  @p-rintz − PR #14).
 * Possibility to toggle file's backup (thanks to @p-rintz − PR #15).
 * Gentoo-specific variables (thanks to @VTimofeenko − PR #22).
 * Ability to specify nft binary path through **nft__bin_location** (thanks to @VTimofeenko − PR #22).
 * Manage Fail2ban in the "systemd way" (thanks to @FinweVI − PR #16).
 * Molecule tests (on Archlinux, Ubuntu, CentOS, Debian and Fedora) (many thanks to @kravietz ! − PR #23).
 * Support for Debian Bullseye (everything should now works fine).
 ### Removed
 * Remove everything related to **in_udp_accept** (see conversation in
  [Github PR #13](https://github.com/ipr-cnrs/nftables/pull/13)).
  Cause it was empty by default and the role currently doesn't manage it very
  well. Take a look to new examples in README.md to find your preferred solution
  (re-adding it, new simple/multi-ports filter rule,…).
 ### Fixed
 * Ansible-lint: Fix line longer than 160 chars.
 * Start nftables systemd unit earlier (thanks to @kravietz − PR #19).
 * Ensure to disable nftables systemd unit from old target (PR #20).
 * Move systemd "Protect" options for nftables to specific override.conf file (PR #20).
 ## v1.7.0
 ### Features
 * Allow to merge group variables with **nft_merged_groups** (#11 #12).
 ### Enhancements
 * Debug var with **nft_debug** (useful to set up merging group variables).
 * Extra example to override default variables.
 ### Fix
 * Add missing ICMPv6 rule.
 ## v1.6.0
 ### Features
 * Able to manage a new NAT table (with prerouting and postrouting chains).
 * Block ipv6 multicast by default.
 ### Enhancements
 * Clean tasks name and comments in tasks/main.yml file.
 * Order and clean comments in defaults/main.yml file.
 * Reload rules instead of restart to avoid to loose rulebase due to invalid syntax (#3 Github).
 ### Fix
 * Fix deprecation warning with ansible 2.7: Invoking "apt" only once while
  using a loop via squash_actions is deprecated.
 * Turn nft_old_pkg_list into a list.
 * Add libiptc0 (iptables dependency) to the list of old package to remove.
 * The 10 minutes delay at first run (#1)!
 ## v1.5.0
 ### Enhancements
 * Add a variable to disable "Protect" instructions in systemd unit.
 * Improve vars description/comments in default/main.yml.
 * Add a variable to manage custom content (table, include,…).
 ## v1.4.1
 ### Fix
 * Set empty dependencies line to fix Galaxy warning.
 * Add possibility to restart Fail2ban service.
 * Use to_nice_json to manage packages list.
 * Fix E405 Remote package tasks should have a retry.
 ## v1.4.0
 ### Enhancements
 * Set a variable to enable/disable the support of Nftables.
 * Move two tasks in systemd handler (try to fix #1).
 * Add a additionnal level for all vars for all hosts (group_vars/all).
 ### Fix
 * Deprecation warning for state "installed".
 * The role now might require Ansible 2.5 (available in Debian Stable backports).
 ## v1.3.1
 ### Fix
 * Reload systemd daemons only if unit file change.
-## v1.3.0
+## v1.3
 ### Features
 * Provide the systemd unit.
@ -111,7 +23,7 @@
 ### Features
 * Allow icmpv6 outgoing traffic.
-## v1.2.0
+## v1.2
 ### Features
 * Ensure to remove old packages (iptables,…).
@ -119,7 +31,7 @@
 ### Fixes
 * Ensure to create the the directory to store the differents configuration files (/etc/nftables.d).
-## v1.1.0
+## v1.1
 ### Features
 * Manage nftables service at startup.
@ -132,7 +44,7 @@
 * Remove DHCP incoming packets. The connection is started by the host, don't need incoming rule.
 * Allow outgoing OpenPGP HTTP requests.
-## v1.0.0
+## v1.0
 ### Features
 * Install `nftables` package for Debian based distros.
--- a/README.md
+++ b/README.md
@ -1,104 +1,68 @@
 # Nftables
 1. [Overview](#overview)
-1. [Role Variables](#role-variables)
+2. [Role Variables](#role-variables)
     * [OS Specific Variables](#os-specific-variables)
     * [Rules Dictionaries](#rules-dictionaries)
-1. [Examples](#examples)
+3. [Example Playbook](#example-playbook)
-     * [With playbooks](#with-playbooks)
+4. [Known Issue](#known-issue)
-     * [With group_vars and host_vars](#with-group_vars-and-host_vars)
+5. [Configuration](#configuration)
-1. [Configuration](#configuration)
+6. [Development](#development)
-     * [Fail2ban integration](#fail2ban-integration)
+7. [License](#license)
-1. [Development](#development)
+8. [Author Information](#author-information)
 1. [License](#license)
 1. [Author Information](#author-information)
 ## Overview
 A role to manage Nftables rules and packages.
-Highly inspired by [Mike Gleason firewall role][mikegleasonjr firewall github]
+Highly inspired by [Mike Gleason firewall role][mikegleasonjr firewall github] (3 levels of rules definition and template), thanks ! I hope i haven't complexify his philosophy… ^^
 (3 levels of rules definition and template), thanks ! I hope i haven't
 complexify his philosophy… (I'm pretty sure, i now did complexify it :D) ^^
 ## Role Variables
-* **nft_enabled** : Enable or disable support for Nftables [default : `true`].
+* **nft_pkg_manage** : If `nftables` package(s) should be managed with this role [default : `true`].
-* **nft_pkg_state** : State of new `nftables` package(s) [default : `present`].
+* **nft_pkg_state** : State of new `nftables` package(s) [default : `installed`].
 * **nft_old_pkg_list** : The list of useless packages to remove (such as Iptables,…) [default : `iptables`].
 * **nft_old_pkg_state** : State of old package(s) [default : `absent`].
 * **nft_old_pkg_manage** : If old package(s) should be managed with this role [default : `true`].
 * **nft_conf_dir_path** : Directory to store the differents Nftables configuration files [default : `/etc/nftables.d`].
 * **nft_main_conf_path** : Main configuration file loaded by systemd unit [default : `/etc/nftables.conf`].
 * **nft_main_conf_content** : Template used to generate the previous main configuration file [default : `etc/nftables.conf.j2`].
-* **nft_input_conf_path** : Input configuration file include in main configuration file [default : `{{ nft_conf_dir_path }}/filter-input.nft`].
+* **nft_input_conf_path** : Input configuration file include in main configuration file [default : `{{ nft_conf_dir_path }}filter-input.nft`].
 * **nft_input_conf_content** : Template used to generate the previous input configuration file [default : `etc/nftables.d/filter-input.nft.j2`].
-* **nft_output_conf_path** : Output configuration file include in main configuration file [default : `{{ nft_conf_dir_path }}/filter-output.nft`].
+* **nft_output_conf_path** : Output configuration file include in main configuration file [default : `{{ nft_conf_dir_path }}filter-output.nft`].
 * **nft_output_conf_content** : Template used to generate the previous output configuration file [default : `etc/nftables.d/filter-output.nft.j2`].
-* **nft_forward_conf_path** : forward configuration file include in main configuration file [default : `{{ nft_conf_dir_path }}/filter-forward.nft`].
+* **nft_define_conf_path** : Vars definition file include in main configuration file [default : `{{ nft_conf_dir_path }}defines.nft`].
 * **nft_forward_conf_content** : Template used to generate the previous forward configuration file [default : `etc/nftables.d/filter-forward.nft.j2`].
 * **nft_define_conf_path** : Vars definition file include in main configuration file [default : `{{ nft_conf_dir_path }}/defines.nft`].
 * **nft_define_conf_content** : Template used to generate the previous vars definition file [default : `etc/nftables.d/defines.nft.j2`].
-* **nft_sets_conf_path** : Sets and maps definition file include in main configuration file [default : `{{ nft_conf_dir_path }}/sets.nft`].
+* **nft_sets_conf_path** : Sets and maps definition file include in main configuration file [default : `{{ nft_conf_dir_path }}sets.nft`].
 * **nft_sets_conf_content** : Template used to generate the previous sets and maps definition file [default : `etc/nftables.d/sets.nft.j2`].
 * **nft_global_default_rules** : Set default rules for `global` chain. Other chains will jump to `global` before apply their specific rules.
-* **nft_global_rules** : You can add `global` rules or override those defined by **nft_global_default_rules** for all hosts.
+* **nft_global_group_rules** : You can add `global` rules or override those defined by **nft_global_default_rules** for a group.
-* **nft_global_group_rules** : You can add `global` rules or override those defined by **nft_global_default_rules** and **nft_global_rules** for a group.
+* **nft_global_host_rules:** : Hosts can also add or override `global` rules.
 * **nft_global_host_rules** : Hosts can also add or override all previours rules.
 * **nft__custom_content** : Custom content (tables, include,…) to add in Nftables configuration [default : `''`].
 * **nft_input_default_rules** : Set default rules for `input` chain.
-* **nft_input_rules** : You can add `input` rules or override those defined by **nft_input_default_rules** for all hosts.
+* **nft_input_group_rules** : You can add `input` rules or override those defined by **nft_input_default_rules** for a group.
-* **nft_input_group_rules** : You can add `input` rules or override those defined by **nft_input_default_rules** and **nft_input_rules** for a group.
+* **nft_input_host_rules:** : Hosts can also add or override `input` rules.
 * **nft_input_host_rules:** : Hosts can also add or override all previous `input` rules.
 * **nft_output_default_rules** : Set default rules for `output` chain.
-* **nft_output_rules** : You can add `output` rules or override those defined by **nft_output_default_rules** for all hosts.
+* **nft_output_group_rules** : You can add `output` rules or override those defined by **nft_output_default_rules** for a group.
-* **nft_output_group_rules** : You can add `output` rules or override those defined by **nft_output_default_rules** and **nft_output_rules** for a group.
+* **nft_output_host_rules:** : Hosts can also add or override `output` rules.
 * **nft_output_host_rules** : Hosts can also add or override all previous `output` rules.
 * **nft_forward_default_rules** : Set default rules for `forward` chain.
 * **nft_forward_rules** : You can add `forward` rules or override those defined by **nft_forward_default_rules** for all hosts.
 * **nft_forward_group_rules** : You can add `forward` rules or override those defined by **nft_forward_default_rules** and **nft_forward_rules** for a group.
 * **nft_forward_host_rules** : Hosts can also add or override all previous `forward` rules.
 * **nft__forward_table_manage** : If the forward table should be managed [default : `False`].
 * **nft__nat_table_manage** : If the nat table should be managed [default : `False`].
 * **nft__nat_default_prerouting_rules** : Set default rules for `prerouting` chain of **nat** table.
 * **nft__nat_prerouting_rules** : Set rules for `prerouting` chain of **nat** table for all hosts in the Ansible inventory.
 * **nft__nat_group_prerouting_rules** : Set rules for `prerouting` chain of **nat** table for hosts in specific Ansible inventory group.
 * **nft__nat_host_prerouting_rules** : Set rules for `prerouting` chain of **nat** table for specific hosts the Ansible inventory.
 * **nft__nat_prerouting_conf_path** : Prerouting configuration file include in the main configuration [default : `{{ nft_conf_dir_path }}/nat-prerouting.nft`].
 * **nft__nat_prerouting_conf_content** : Template used to generate the previous prerouting configuration file [default : `etc/nftables.d/nat-prerouting.nft.j2`].
 * **nft__nat_default_postrouting_rules** : Set default rules for `postrouting` chain of **nat** table.
 * **nft__nat_postrouting_rules** : Set rules for `postrouting` chain of **nat** table for all hosts in the Ansible inventory.
 * **nft__nat_group_postrouting_rules** : Set rules for `postrouting` chain of **nat** table for hosts in specific Ansible inventory group.
 * **nft__nat_host_postrouting_rules** : Set rules for `postrouting` chain of **nat** table for specific hosts the Ansible inventory.
 * **nft__nat_postrouting_conf_path** : postrouting configuration file include in the main configuration [default : `{{ nft_conf_dir_path }}/nat-postrouting.nft`].
 * **nft__nat_postrouting_conf_content** : Template used to generate the previous postrouting configuration file [default : `etc/nftables.d/nat-postrouting.nft.j2`].
 * **nft_define_default** : Set default vars available in all rules.
-* **nft_define** : You can add vars or override those defined by **nft_define_default** for all hosts.
+* **nft_define_group** : You can add vars or override those defined by **nft_define_default** for groups.
-* **nft_define_group** : You can add vars or override those defined by **nft_define_default** and **nft_define** for a group.
+* **nft_define_host** : You can add or override existant vars.
 * **nft_define_host** : You can add or override all previous vars.
 * **nft_service_manage** : If `nftables` service should be managed with this role [default : `true`].
 * **nft_service_name** : `nftables` service name [default : `nftables`].
 * **nft_service_enabled** : Set `nftables` service available at startup [default : `true`].
 * **nft__service_protect** : If systemd unit should protect system and home [default : `true`].
 * **nft_merged_groups** : If variables from the hosts Ansible groups should be merged [default : `false`].
 * **nft_merged_groups_dir** : The dictionary where the nftables group rules, named like the Ansible groups, are located in [default : `vars/`].
 * **nft_debug** : Toggle more verbose output on/off. [default: 'false'].
 ### OS Specific Variables
 Please see default value by Operating System file in [vars][vars directory] directory.
 * **nft_pkg_list** : The list of package(s) to provide `nftables`.
 * **nft__bin_location** : Path to `nftables` executable. [default : `/usr/sbin/nft`]
 ### Rules Dictionaries
 Each type of rules dictionaries will be merged and rules will be applied in the alphabetical order of the keys (the reason to use 000 to 999 as prefix). So :
  * **nft_*_default_rules** : Define default rules for all nodes. You can define it in `group_vars/all`.
-  * **nft_*_rules** : Can add rules and override those defined by **nft_*_default_rules**. You can define it in `group_vars/all`.
+  * **nft_*_group_rules** : Can add rules and override those defined by **nft_*_default_rules**. You can define it in `group_vars/webservers`.
-  * **nft_*_group_rules** : Can add rules and override those defined by **nft_*_default_rules** and **nft_*_rules**. You can define it in `group_vars/webservers`.
+  * **nft_*_host_rules** : Can add rules and override those define by **nft_*_default_rules** and **nft_*_group_rules**. You can define it in `host_vars/www.local.domain`.
    * If 'nft_merged_groups' is set to true, multiple group rules from the ansible groups will also be merged together.
  * **nft_*_host_rules** : Can add rules and override those define by **nft_*_default_rules**, **nft_*_group_rules** and **nft_*_rules**. You can define it in `host_vars/www.local.domain`.
 `defaults/main.yml`:
@ -108,9 +72,6 @@ nft_global_default_rules:
  005 state management:
    - ct state established,related accept
    - ct state invalid drop
 nft_global_rules: {}
 nft_merged_groups: false
 nft_merged_groups_dir: vars/
 nft_global_group_rules: {}
 nft_global_host_rules: {}
@ -123,9 +84,10 @@ nft_input_default_rules:
    - ip daddr @blackhole counter drop
  015 localhost:
    - iif lo accept
  200 input udp accepted:
    - udp dport @in_udp_accept ct state new accept
  210 input tcp accepted:
    - tcp dport @in_tcp_accept ct state new accept
 nft_input_rules: {}
 nft_input_group_rules: {}
 nft_input_host_rules: {}
@ -138,12 +100,10 @@ nft_output_default_rules:
    - oif lo accept
  050 icmp:
    - ip protocol icmp accept
    - ip6 nexthdr icmpv6 counter accept
  200 output udp accepted:
    - udp dport @out_udp_accept ct state new accept
  210 output tcp accepted:
    - tcp dport @out_tcp_accept ct state new accept
 nft_output_rules: {}
 nft_output_group_rules: {}
 nft_output_host_rules: {}
@ -156,13 +116,15 @@ nft_define_default:
  input tcp accepted:
    name: in_tcp_accept
    value: '{ ssh }'
  input udp accepted:
    name: in_udp_accept
    value: 'none'
  output tcp accepted:
    name: out_tcp_accept
    value: '{ http, https, hkp }'
  output udp accepted:
    name: out_udp_accept
    value: '{ bootps, domain, ntp }'
 nft_define: {}
 nft_define_group: {}
 nft_define_host: {}
@ -174,13 +136,14 @@ nft_set_default:
  in_tcp_accept:
    - type inet_service; flags interval;
    - elements = $in_tcp_accept
  in_udp_accept:
    - type inet_service; flags interval;
  out_tcp_accept:
    - type inet_service; flags interval;
    - elements = $out_tcp_accept
  out_udp_accept:
    - type inet_service; flags interval;
    - elements = $out_udp_accept
 nft_set: {}
 nft_set_group: {}
 nft_set_host: {}
 ```
@ -238,6 +201,7 @@ table inet filter {
 		jump global
 		ip daddr @blackhole counter packets 0 bytes 0 drop
 		iif "lo" accept
 		udp dport @in_udp_accept ct state new accept
 		tcp dport @in_tcp_accept ct state new accept
 	}
@ -252,268 +216,17 @@ table inet filter {
 }
 ```
-## Examples
+## Example Playbook
-### With playbooks
+* Manage Nftables with defaults vars :
 <details>
  <summary>Manage Nftables with defaults vars (<i>click to expand</i>)</summary>
  <!-- have to be followed by an empty line! -->
 ``` yml
 - hosts: serverXYZ
  roles:
    - role: ipr-cnrs.nftables
 ```
 </details>
-<details>
+* Use default rules with allow incoming ICMP and count dropped input packets :
  <summary>Add a new <b>simple</b> filter rule for incoming traffic (eg. 1 port for UDP/torrent) (<i>click to expand</i>)</summary>
 ``` yml
 - hosts: serverXYZ
  vars:
      nft_input_rules:
        400 input torrent accepted:
          - udp dport 6881 ct state new accept
  roles:
    - role: ipr-cnrs.nftables
 ```
 * **nft_input_group_rules** or **nft_input_host_rules** variables can
  also be used.
 * The weight (`400`) allow to order all merged rules (from
  <b>nft_input_*rules</b> dictionaries).
 * The text following the weight (`input torrent accepted`) is a small
  description that will be added as a comment in **nft_input_conf_path** file
  on the remote host.
 </details>
 <details>
  <summary>Add a new <b>multi-ports</b> filter rule for incoming traffic (eg. TCP/http, https, http-alt,…) (<i>click to expand</i>)</summary>
 ``` yml
 - hosts: serverXYZ
  vars:
      nft_input_rules:
        400 input http accepted:
          - tcp dport { 80, 443, 8080-8082 } ct state new accept
  roles:
    - role: ipr-cnrs.nftables
 ```
 * **nft_input_group_rules** or **nft_input_host_rules** variables can
  also be used.
 * The weight (`400`) allow to order all merged rules (from
  <b>nft_input_*rules</b> dictionaries).
 * The text following the weight (`input http accepted`) is a small
  description that will be added as a comment in **nft_input_conf_path** file
 on the remote host.
 * In this case, brackets are useful and define a anonymous set. For a single
  element (port, IP address,…), brackets are overkill and the singleton
  definition is enought.
 </details>
 <details>
  <summary>Add a new rule with a variable (<i>click to expand</i>)</summary>
 Nftables variables can be useful if you define somes generic rules for all hosts
 with such variables (called with **$**) and override variable's value for some
 groups or hosts.
 ``` yml
 - hosts: serverXYZ
  vars:
    - nft_define_group:
        input http accepted:
          desc: HTTP and HTTPS
          name: in_http_accept
          value: '{ 80, 443 }'
      nft_input_group_rules:
        400 input http accepted:
          - tcp dport $in_http_accept ct state new accept
  roles:
    - role: ipr-cnrs.nftables
 ```
 1. Add a new variable with **define** for HTTP ports.
 1. Add a new rule for incoming traffic and use the previous defined variable.
 1. Result of `nft list ruleset` on the remote host will be :
   ``` bash
   table inet filter {
           …
           chain input {
                   …
                   tcp dport { http, https } ct state new accept
                   …
           }
           …
   }
   ```
   * No mention of `$in_http_accept` variable.
 * **nft_define** or **nft_define_host** variables can also be used.
 * **nft_input_rules** or **nft_input_host_rules** variables can also be used.
 * The weight (`400`) allow to order all merged rules (from
  <b>nft_input_*rules</b> dictionaries).
 * The text following the weight (`input http accepted`) is a small description
  that will be added as a comment in **nft_input_conf_path** file on
  the remote host.
 </details>
 <details>
  <summary>Add a new rule with a named set (<i>click to expand</i>)</summary>
 Quite similar to Nftables variables, **named set** can be useful if you define
 somes generic rules and sets (eg. for all hosts) and override only the set in
 some case (eg. for a group or some hosts).
 In addition to variables, it is possible to add content to named sets on the
 fly from the host without completely rewrite the rule.
 ``` yml
 - hosts: serverXYZ
  vars:
      nft_set_group:
        in_udp_accept:
          - type inet_service; flags interval;
          - elements = { 6881-6887, 6889 }
      nft_input_group_rules:
        200 input udp accepted:
          - udp dport @in_udp_accept ct state new accept
  roles:
    - role: ipr-cnrs.nftables
 ```
 1. Add a new named set with <b>nft_set_group</b> dictionary (eg. for torrent ports).
 1. Add a new rule for incoming traffic and use the previous defined set.
 1. On the remote host, if you try to add a port to this set : `nft add element inet filter in_udp_accept \{ 6999 \}`
 1. Result of `nft list ruleset` on the remote host will now be :
   ``` bash
   table inet filter {
           …
           set in_udp_accept {
                   type inet_service
                   flags interval
                   elements = { 6881-6887, 6889, 6999 }
           }
           chain input {
                   …
                   udp dport @in_udp_accept ct state new accept
                   …
           }
           …
   }
   ```
 * **nft_set** or **nft_set_host** variables can also be used.
 * **nft_input_rules** or **nft_input_host_rules** variables can also be used.
 * The weight (`200`) allow to order all merged rules (from
  <b>nft_input_*rules</b> dictionaries).
 * The text following the weight (`input upd accepted`) is a small description
  that will be added as a comment in **nft_input_conf_path** file on
  the remote host.
 </details>
 <details>
  <summary>Override a default rule with 2 new rules (<i>click to expand</i>)</summary>
 ``` yml
 - hosts: serverXYZ
  vars:
      nft_input_host_rules:
        050 icmp:⏎
          - ip protocol icmp  ip saddr != 192.168.0.0/24  counter  drop⏎
          - ip protocol icmp  icmp type echo-request  ip length <= 84  counter  limit rate 10/minute  accept
  roles:
    - role: ipr-cnrs.nftables
 ```
 1. Get rule description from `defaults/main.yml` file (eg. `050 icmp`).
 1. Drop any ICMP request that doesn't come from 192.168.0.0 network.
 1. Ensure the request is less or equal to 84 bytes and set up a limit
   to 10 requests per minute.
 * **nft_input_rules** or **nft_input_group_rules** variables can also be used.
 * The weight (`050`) allow to order all merged rules (from
  <b>nft_input_*rules</b> dictionaries).
 * The text following the weight (`icmp`) is a small description that
  will be added as a comment in **nft_input_conf_path** file on
  the remote host.
 </details>
 <details>
  <summary>Override some of the default defined sets (<i>click to expand</i>)</summary>
 ``` yml
 - hosts: serverXYZ
  vars:
    - nft_define:
      input tcp accepted:
        desc: Custom SSH port and torrent
        name: in_tcp_accept
        value: '{ 2201, 6881 }'
  roles:
    - role: ipr-cnrs.nftables
 ```
 1. Get item name (eg. `input tcp accepted`) and variable name
   (eg. `in_tcp_accept`) from `defaults/main.yml` file.
 1. Set a new value (eg. `'{ 2201, 6881 }'`).
 1. You can add a `desc` attribute that will be set as a comment in
   **nft_input_conf_path** file on the remote host.
 * **nft_define_group** or **nft_define_host** variables can also be used.
 </details>
 <details>
  <summary>Override all default rules (eg. for outgoing traffic) (<i>click to expand</i>)</summary>
 If the default rules are too permissive, if you already override most of them,…
 In some case, i guess, it can be interesting to redefine the default variable :
 ``` yml
 - hosts: serverXYZ
  vars:
      nft_output_default_rules:
        000 policy:
          - type filter hook output priority 0; policy drop;
        005 state management:
          - ct state established,related accept
          - ct state invalid drop
        015 localhost:
          - oif lo accept
        050 my rule for XXX hosts and services:
          - tcp dport 2000  ip saddr { xxx.xxx.xxx.xxx, yyy.yyy.yyy.yyy }  ct state new  accept
        250 reset-ssh:  # allow the host to reset SSH connections to avoid 10 min delay from Ansible controller
          - tcp sport ssh tcp flags { rst, psh | ack } counter accept
  roles:
    - role: ipr-cnrs.nftables
 ```
 * At least, don't forget to :
  1. set a default `policy`.
  1. manage already `established` state.
  1. accept `rst, psh | ack` flags for **ssh** to avoid a 10 minutes delay at
     the first run of this Nftables role (see #1).
 * Then add your own rules with the wanted weight to order all merged rules (from
  <b>nft_output_*rules</b> dictionaries) and descriptions.
 </details>
 <details>
  <summary>Remove a default rule (<i>click to expand</i>)</summary>
 ``` yml
 - hosts: serverXYZ
  vars:
      nft_output_host_rules:
        210 output tcp accepted:
          -
  roles:
    - role: ipr-cnrs.nftables
 ```
 1. Get rule description from `defaults/main.yml` file (`210 output tcp accepted`).
 1. The default policy for outgoing traffic (**drop**) will now be applied to
   ports defined in **out_tcp_accept** variable. Be sure of what you are doing.
 1. The rule will no longer be present in `nft list ruleset` result, just a
   comment will remain (`210 output tcp accepted`) in **nft_output_conf_path**
   file on the remote host.
 * **nft_output_rules** or **nft_output_group_rules** variables can also be used.
 * The weight (`210`) allow to order all merged rules (from
  <b>nft_output_*rules</b> dictionaries).
 </details>
 ### With group_vars and host_vars
 <details>
  <summary>Use default rules and allow, for <b>first_group</b>, incoming ICMP and count both ICMP and default policy (<b>drop</b>) packets (<i>click to expand</i>)</summary>
 `group_vars/first_group` :
@ -524,70 +237,33 @@ nft_input_group_rules:
  999 count policy packet:
    - counter
 ```
 </details>
-<details>
+## Known Issue
  <summary>Use merged group rules from multiple ansible groups (<i>click to expand</i>)</summary>
-1. Enable to merge group's variables :
+* The first run of a playbook may be long (~8 minutes) just after the (re)start of `nftables` service.
-    ``` yml
+  * Probably due to Ansible/OpenSSH Specific default Settings, the outgoing connections for Ansible/SSH will be blocked :
    - hosts: serverXYZ
      vars:
        nft_merged_groups: true
        nft_merged_groups_dir: vars/
      roles:
        - role: ipr-cnrs.nftables
 ```
-1. Put extra rules inside the "vars" folder named after your ansible groups for serverXYZ :
+trace id d7c3a8dc inet filter output packet: oif "ens18" ip saddr REMOTE_HOST ip daddr MY_HOST ip dscp 0x02 ip ecn not-ect ip ttl 64 ip id 56799 ip length 420 tcp sport ssh tcp dport 53922 tcp flags == 0x18 tcp window 1452
   * `vars/first_group` :
       ``` yaml
       nft_input_group_rules:
         020 icmp:
           - ip protocol icmp icmp type echo-request ip length <= 84 counter limit rate 1/minute accept
         999 count policy packet:
           - counter
 ```
-   * `vars/second_group` :
+  * This only happen at the first run on new clean host. The next runs will works without any delay.
       ``` yaml
       nft_input_group_rules:
         021 LAN:
           - iif eth0 accept
       ```
 1. These rulesets, from the two groups, will be merged if the host is a member
   of these groups.
 </details>
 ## Configuration
 This role will :
 * Install `nftables` on the system.
 * Enable `nftables` service by default at startup.
-* Generate a default configuration file which include all following files and
+* Generate a default configuration file which include all following files and loaded by systemd unit.
  loaded by systemd unit.
 * Generate input and output rules files include called by the main configuration file.
 * Generate vars in a file and sets and maps in another file.
-* (re)Start `nftables` service at first run.
+* Restart `nftables` service.
 * Reload `nftables` service at next runs to avoid to let the host without firewall
  rules due to invalid syntax.
 ### Fail2ban integration
 Before Debian Bullseye, systemd unit for Fail2ban doesn't come with a decent
 integration with Nftables.
 So this role will create override file for `fail2ban` unit, even if it's not
 (yet) available on the host, in order to :
 * Start `fail2ban` unit after `nftables`.
 * Restart `fail2ban` unit when `nftables` unit restart.
 ## Development
-This source code comes from our [Gitea instance][nftables source] and the
+This source code comes from our [Gogs instance][nftables source] and the [Github repo][nftables github] exist just to be able to send the role to Ansible Galaxy…
 [Github repo][nftables github] exist just to be able to send the role to Ansible
 Galaxy…
 But feel free to send issue/PR here :)
-Thanks to this [hook][gogs to github hook], Github automatically got updates from our [Gitea instance][nftables source] :)
+Thanks to this [hook][gogs to github hook], Github automatically got updates from our [Gogs instance][nftables source] :)
 ## License
@ -596,11 +272,11 @@ Thanks to this [hook][gogs to github hook], Github automatically got updates fro
 ## Author Information
 Jérémy Gardais
-* Source : [on IPR's Gitea][nftables source]
+* Source : [on IPR's Gogs][nftables source]
 * [IPR][ipr website] (Institut de Physique de Rennes)
 [gogs to github hook]: https://stackoverflow.com/a/21998477
-[nftables source]: https://git.ipr.univ-rennes.fr/cellinfo/ansible.nftables
+[nftables source]: https://git.ipr.univ-rennes1.fr/cellinfo/ansible.nftables
 [nftables github]: https://github.com/ipr-cnrs/nftables
 [wtfpl website]: http://www.wtfpl.net/about/
 [ipr website]: https://ipr.univ-rennes1.fr/
--- a/defaults/main.yml
+++ b/defaults/main.yml
@ -1,266 +1,34 @@
 ---
-# .. vim: foldmarker=[[[,]]]:foldmethod=marker
+# defaults file for nftables
 #
 # ipr-cnrs.nftables default variables
 # ===================================
-# Packages and installation [[[
+# packages
-# -----------------------------
+nft_pkg_manage: true
-
+nft_pkg_state: 'installed'
-# .. envvar:: nft_enabled [[[
+nft_old_pkg_list: 'iptables'
 #
 # Enable or disable support for Nftables on a given host. Disabling this
 # option does not remove existing installation and configuration.
 #
 nft_enabled: true
                                                                   # ]]]
 # .. envvar:: nft_pkg_state [[[
 #
 # What is the desired state for Nftables packages ? Possible options :
 #
 # ``present``
 #   Default. Nftables packages will be installed.
 #
 # ``absent``
 #   Ensure to remove Nftables related packages.
 nft_pkg_state: 'present'
                                                                   # ]]]
 # .. envvar:: nft_old_pkg_list [[[
 #
 # List of old Iptables packages to remove to avoid duplicate firewall.
 #
 nft_old_pkg_list:
  - 'iptables'
  - 'libiptc0'
                                                                   # ]]]
 # .. envvar:: nft_old_pkg_state [[[
 #
 # What is the desired state for Iptables packages ? Possible options :
 #
 # ``absent``
 #   Default. Ensure to remove Iptables related packages.
 #
 # ``present``
 #   Iptables packages will be kept.
 nft_old_pkg_state: 'absent'
                                                                   # ]]]
 # .. envvar:: nft_old_pkg_manage [[[
 #
 # If the old Iptables packages should be managed ? Possible options :
 #
 # ``true``
 #   Default. Ensure to apply the required state for Iptables related packages.
 #
 # ``false``
 #   Iptables packages will not be touched.
 nft_old_pkg_manage: true
                                                                   # ]]]
                                                                   # ]]]
 # Nftables global rules [[[
 # -------------------------
-# .. envvar:: nft_global_default_rules [[[
+# files
-#
+nft_conf_dir_path: '/etc/nftables.d'
-# List of global rules (applied on all tables) to configure for all hosts
+nft_main_conf_path: '/etc/nftables.conf'
-# inherited from this role.
+nft_main_conf_content: 'etc/nftables.conf.j2'
 nft_input_conf_path: '{{ nft_conf_dir_path }}/filter-input.nft'
 nft_input_conf_content: 'etc/nftables.d/filter-input.nft.j2'
 nft_output_conf_path: '{{ nft_conf_dir_path }}/filter-output.nft'
 nft_output_conf_content: 'etc/nftables.d/filter-output.nft.j2'
 nft_define_conf_path: '{{ nft_conf_dir_path }}/defines.nft'
 nft_define_conf_content: 'etc/nftables.d/defines.nft.j2'
 nft_set_conf_path: '{{ nft_conf_dir_path }}/sets.nft'
 nft_set_conf_content: 'etc/nftables.d/sets.nft.j2'
 # rules
 nft_global_default_rules:
  005 state management:
    - ct state established,related accept
    - ct state invalid drop
                                                                   # ]]]
 # .. envvar:: nft_global_rules [[[
 #
 # List of global rules (applied on all tables) to configure for all hosts
 # in the Ansible inventory.
 nft_global_rules: {}
                                                                   # ]]]
 # .. envvar:: nft_merged_groups [[[
 #
 # Enable or disable the ability to merge multiple firewall group variables
 nft_merged_groups: false
                                                                   # ]]]
 # .. envvar:: nft_merged_groups_dir [[[
 #
 # The directory to read the group firewall rules from.
 # Relative to the playbook directory.
 nft_merged_groups_dir: vars/
                                                                   # ]]]
 # .. envvar:: nft_global_group_rules [[[
 #
 # List of global rules (applied on all tables) to configure for hosts in
 # specific Ansible inventory group.
 nft_global_group_rules: {}
                                                                   # ]]]
 # .. envvar:: nft_global_host_rules [[[
 #
 # List of global rules (applied on all tables) to configure for specific hosts
 # in the Ansible inventory.
 nft_global_host_rules: {}
                                                                   # ]]]
 # .. envvar:: nft__custom_content [[[
 #
 # Custom content (tables, include,…) to add in Nftables configuration.
 nft__custom_content: ''
                                                                   # ]]]
 # .. envvar:: nft_custom_includes [[[
 #
 # Custom includes to add into the main Nftables filter configuration.
 nft_custom_includes: ''
                                                                   # ]]]
 # .. envvar:: nft_conf_dir_path [[[
 #
 # Path to the sub directory for Nftables configuration files.
 #
 # Should be an absolut path and this var will be used in all *_path vars.
 nft_conf_dir_path: '/etc/nftables.d'
                                                                   # ]]]
 # .. envvar:: nft_main_conf_path [[[
 #
 # Path to the main configuration file called by the Systemd Nftables service.
 #
 # Should be an absolut path.
 nft_main_conf_path: '/etc/nftables.conf'
                                                                   # ]]]
 # .. envvar:: nft_main_conf_content [[[
 #
 # Template used to provide the previous main configuration file.
 #
 # Must be a relative path from default/ directory of this role or from your
 # Ansible inventory directory.
 nft_main_conf_content: 'etc/nftables.conf.j2'
                                                                   # ]]]
                                                                   # ]]]
 # Nftables vars definition [[[
 # ----------------------------
 #
 # These lists allow to define some vars that can be used in nftables rules.
 # See the official Nftables wiki page for more informations and examples :
 # https://wiki.nftables.org/wiki-nftables/index.php/Scripting#Defining_variables
 # .. envvar:: nft_define_default [[[
 #
 # List of vars definition to configure for all hosts inherited from this role.
 nft_define_default:
  broadcast and multicast:
    desc: 'broadcast and multicast'
    name: badcast_addr
    value: '{ 255.255.255.255, 224.0.0.1, 224.0.0.251 }'
  ip6 broadcast and multicast:
    desc: 'broadcast and multicast'
    name: ip6_badcast_addr
    value: '{ ff02::16 }'
  input tcp accepted:
    name: in_tcp_accept
    value: '{ ssh }'
  output tcp accepted:
    name: out_tcp_accept
    value: '{ http, https, hkp }'
  output udp accepted:
    name: out_udp_accept
    value: '{ bootps, domain, ntp }'
                                                                   # ]]]
 # .. envvar:: nft_define [[[
 #
 # List of vars definition to configure for all hosts in the Ansible inventory.
 nft_define: {}
                                                                   # ]]]
 # .. envvar:: nft_define_group [[[
 #
 # List of vars definition to configure for hosts in specific
 # Ansible inventory group.
 nft_define_group: {}
                                                                   # ]]]
 # .. envvar:: nft_define_host [[[
 #
 # List of vars definition to configure for specific hosts
 # in the Ansible inventory.
 nft_define_host: {}
                                                                   # ]]]
 # .. envvar:: nft_define_conf_path [[[
 #
 # Path to the defined vars file to include in the main configuration file
 # in order to use the previous defined lists.
 #
 # Should include the '{{ nft_conf_dir_path }}' var or be an absolut path.
 nft_define_conf_path: '{{ nft_conf_dir_path }}/defines.nft'
                                                                   # ]]]
 # .. envvar:: nft_define_conf_content [[[
 #
 # Template used to provide the previous defined vars file.
 #
 # Must be a relative path from default/ directory of this role or from your
 # Ansible inventory directory.
 nft_define_conf_content: 'etc/nftables.d/defines.nft.j2'
                                                                   # ]]]
                                                                   # ]]]
 # Nftables sets definition [[[
 # ----------------------------
 #
 # These "set" lists allow to define sets that can be used in Nftables rules.
 # See the official Nftables wiki page for more informations and examples :
 # https://wiki.nftables.org/wiki-nftables/index.php/Sets
 # .. envvar:: nft_define_default [[[
 #
 # List of sets to configure for all hosts inherited from this role.
 nft_set_default:
  blackhole:
    - type ipv4_addr;
    - elements = $badcast_addr
  ip6blackhole:
    - type ipv6_addr;
    - elements = $ip6_badcast_addr
  in_tcp_accept:
    - type inet_service; flags interval;
    - elements = $in_tcp_accept
  out_tcp_accept:
    - type inet_service; flags interval;
    - elements = $out_tcp_accept
  out_udp_accept:
    - type inet_service; flags interval;
    - elements = $out_udp_accept
                                                                   # ]]]
 # .. envvar:: nft_set [[[
 #
 # List of sets to configure for all hosts in the Ansible inventory.
 nft_set: {}
                                                                   # ]]]
 # .. envvar:: nft_set_group [[[
 #
 # List of sets to configure for hosts in specific Ansible inventory group.
 nft_set_group: {}
                                                                   # ]]]
 # .. envvar:: nft_set_host [[[
 #
 # List of sets to configure for specific hosts in the Ansible inventory.
 nft_set_host: {}
                                                                   # ]]]
 # .. envvar:: nft_set_conf_path [[[
 #
 # Path to the "sets" file to include in the main configuration file
 # in order to use the previous defined lists.
 #
 # Should include the '{{ nft_conf_dir_path }}' var or be an absolut path.
 nft_set_conf_path: '{{ nft_conf_dir_path }}/sets.nft'
                                                                   # ]]]
 # .. envvar:: nft_set_conf_content [[[
 #
 # Template used to provide the previous "sets" file.
 #
 # Must be a relative path from default/ directory of this role or from your
 # Ansible inventory directory.
 nft_set_conf_content: 'etc/nftables.d/sets.nft.j2'
                                                                   # ]]]
                                                                   # ]]]
 # inet filter table rules [[[
 # ---------------------------
 #
 # All these rules will be set up in an inet table in order to filter the
 # input and output traffic.
 # .. envvar:: nft_input_default_rules [[[
 #
 # List of input rules to configure for all hosts inherited from this role.
 nft_input_default_rules:
  000 policy:
    - type filter hook input priority 0; policy drop;
@ -268,50 +36,15 @@ nft_input_default_rules:
    - jump global
  010 drop unwanted:
    - ip daddr @blackhole counter drop
  011 drop unwanted ipv6:
    - ip6 daddr @ip6blackhole counter drop
  015 localhost:
    - iif lo accept
-  050 icmp:
+  200 input udp accepted:
-    - meta l4proto {icmp,icmpv6} accept
+    - udp dport @in_udp_accept ct state new accept
  210 input tcp accepted:
    - tcp dport @in_tcp_accept ct state new accept
                                                                   # ]]]
 # .. envvar:: nft_input_rules [[[
 #
 # List of input rules to configure for all hosts in the Ansible inventory.
 nft_input_rules: {}
                                                                   # ]]]
 # .. envvar:: nft_input_group_rules [[[
 #
 # List of input rules to configure for hosts in specific Ansible inventory group.
 nft_input_group_rules: {}
                                                                   # ]]]
 # .. envvar:: nft_input_host_rules [[[
 #
 # List of input rules to configure for specific hosts in the Ansible inventory.
 nft_input_host_rules: {}
                                                                   # ]]]
 # .. envvar:: nft_input_conf_path [[[
 #
 # Path to the input rules file for the filter table to include in the main
 # configuration file in order to use the previous defined lists.
 #
 # Should include the '{{ nft_conf_dir_path }}' var or be an absolut path.
 nft_input_conf_path: '{{ nft_conf_dir_path }}/filter-input.nft'
                                                                   # ]]]
 # .. envvar:: nft_input_conf_content [[[
 #
 # Template used to provide the previous input rules file.
 #
 # Must be a relative path from default/ directory of this role or from your
 # Ansible inventory directory.
 nft_input_conf_content: 'etc/nftables.d/filter-input.nft.j2'
                                                                   # ]]]
 # .. envvar:: nft_output_default_rules [[[
 #
 # List of output rules to configure for all hosts inherited from this role.
 nft_output_default_rules:
  000 policy:
    - type filter hook output priority 0; policy drop;
@ -326,296 +59,52 @@ nft_output_default_rules:
    - udp dport @out_udp_accept ct state new accept
  210 output tcp accepted:
    - tcp dport @out_tcp_accept ct state new accept
  250 reset-ssh:  # allow the host to reset SSH connections to avoid 10 min delay from Ansible controller
    - tcp sport ssh tcp flags { rst, psh | ack } counter accept
                                                                   # ]]]
 # .. envvar:: nft_output_rules [[[
 #
 # List of output rules to configure for all hosts in the Ansible inventory.
 nft_output_rules: {}
                                                                   # ]]]
 # .. envvar:: nft_output_group_rules [[[
 #
 # List of output rules to configure for hosts in specific Ansible inventory group.
 nft_output_group_rules: {}
                                                                   # ]]]
 # .. envvar:: nft_output_host_rules [[[
 #
 # List of output rules to configure for specific hosts in the Ansible inventory.
 nft_output_host_rules: {}
                                                                   # ]]]
 # .. envvar:: nft_output_conf_path [[[
 #
 # Path to the output rules file for the filter table to include in the main
 # configuration file in order to use the previous defined lists.
 #
 # Should include the '{{ nft_conf_dir_path }}' var or be an absolut path.
 nft_output_conf_path: '{{ nft_conf_dir_path }}/filter-output.nft'
                                                                   # ]]]
 # .. envvar:: nft_output_conf_content [[[
 #
 # Template used to provide the previous output rules file.
 #
 # Must be a relative path from default/ directory of this role or from your
 # Ansible inventory directory.
 nft_output_conf_content: 'etc/nftables.d/filter-output.nft.j2'
                                                                   # ]]]
 # .. envvar:: nft__forward_table_manage [[[
 #
 # If the forward table should be managed ? Possible options are :
 #
 # ``False``
 #   Default. The forward table is not managed and rules will not be added.
 #
 # ``True``
 #   Add the forwarding rules that follow.
 nft__forward_table_manage: false
                                                                   # ]]]
 # .. envvar:: nft_forward_default_rules [[[
 #
 # List of forward rules to configure for all hosts inherited from this role.
 nft_forward_default_rules:
  000 policy:
    - type filter hook forward priority 0; policy drop;
  005 global:
    - jump global
                                                                   # ]]]
 # .. envvar:: nft_forward_rules [[[
 #
 # List of forward rules to configure for all hosts in the Ansible inventory.
 nft_forward_rules: {}
                                                                   # ]]]
 # .. envvar:: nft_forward_group_rules [[[
 #
 # List of forward rules to configure for hosts in specific Ansible inventory group.
 nft_forward_group_rules: {}
                                                                   # ]]]
 # .. envvar:: nft_forward_host_rules [[[
 #
 # List of forward rules to configure for specific hosts in the Ansible inventory.
 nft_forward_host_rules: {}
                                                                   # ]]]
 # .. envvar:: nft_forward_conf_path [[[
 #
 # Path to the forward rules file for the filter table to include in the main
 # configuration file in order to use the previous defined lists.
 #
 # Should include the '{{ nft_conf_dir_path }}' var or be an absolut path.
 nft_forward_conf_path: '{{ nft_conf_dir_path }}/filter-forward.nft'
                                                                   # ]]]
 # .. envvar:: nft_forward_conf_content [[[
 #
 # Template used to provide the previous forward rules file.
 #
 # Must be a relative path from default/ directory of this role or from your
 # Ansible inventory directory.
 nft_forward_conf_content: 'etc/nftables.d/filter-forward.nft.j2'
                                                                   # ]]]
                                                                   # ]]]
 # ip nat table rules [[[
 # ---------------------------
 #
 # All these rules will be set up in an ip table in order to perform some
 # Network Address Translation (NAT).
-# .. envvar:: nft__nat_table_manage [[[
+# define nft vars
-#
+nft_define_default:
-# If the nat table should be managed ? Possible options are :
+  broadcast and multicast:
-#
+    desc: 'broadcast and multicast'
-# ``False``
+    name: badcast_addr
-#   Default. The nat table is not managed and rules will not be added.
+    value: '{ 255.255.255.255, 224.0.0.1, 224.0.0.251 }'
-#
+  input tcp accepted:
-# ``True``
+    name: in_tcp_accept
-#   Add the pre and postrouting rules that follow.
+    value: '{ ssh }'
-nft__nat_table_manage: False
+  input udp accepted:
-                                                                   # ]]]
+    name: in_udp_accept
-# .. envvar:: nft__nat_default_prerouting_rules [[[
+    value: 'none'
-#
+  output tcp accepted:
-# List of prerouting rules to configure for all hosts inherited from this role.
+    name: out_tcp_accept
-nft__nat_default_prerouting_rules:
+    value: '{ http, https, hkp }'
-  000 policy:
+  output udp accepted:
-    - type nat hook prerouting priority 0;
+    name: out_udp_accept
-                                                                   # ]]]
+    value: '{ bootps, domain, ntp }'
-# .. envvar:: nft__nat_prerouting_rules [[[
+nft_define_group: {}
-#
+nft_define_host: {}
 # List of prerouting rules to configure for all hosts in the Ansible inventory.
 nft__nat_prerouting_rules: {}
                                                                   # ]]]
 # .. envvar:: nft__nat_group_prerouting_rules [[[
 #
 # List of prerouting rules to configure for hosts in specific
 # Ansible inventory group.
 nft__nat_group_prerouting_rules: {}
                                                                   # ]]]
 # .. envvar:: nft__nat_host_prerouting_rules [[[
 #
 # List of prerouting rules to configure for specific hosts
 # in the Ansible inventory.
 nft__nat_host_prerouting_rules: {}
                                                                   # ]]]
 # .. envvar:: nft__nat_prerouting_conf_path [[[
 #
 # Path to the prerouting rules file for the nat table to include in the main
 # configuration file in order to use the previous defined lists.
 #
 # Should include the '{{ nft_conf_dir_path }}' var or be an absolut path.
 nft__nat_prerouting_conf_path: '{{ nft_conf_dir_path }}/nat-prerouting.nft'
                                                                   # ]]]
 # .. envvar:: nft__nat_prerouting_conf_content [[[
 #
 # Template used to provide the previous prerouting rules file.
 #
 # Must be a relative path from default/ directory of this role or from your
 # Ansible inventory directory.
 nft__nat_prerouting_conf_content: 'etc/nftables.d/nat-prerouting.nft.j2'
                                                                   # ]]]
-# .. envvar:: nft__nat_default_postrouting_rules [[[
+# sets and maps
-#
+nft_set_default:
-# List of postrouting rules to configure for all hosts inherited from this role.
+  blackhole:
-nft__nat_default_postrouting_rules:
+    - type ipv4_addr;
-  000 policy:
+    - elements = $badcast_addr
-    - type nat hook postrouting priority 100;
+  in_tcp_accept:
-                                                                   # ]]]
+    - type inet_service; flags interval;
-# .. envvar:: nft__nat_postrouting_rules [[[
+    - elements = $in_tcp_accept
-#
+  in_udp_accept:
-# List of postrouting rules to configure for all hosts in the Ansible inventory.
+    - type inet_service; flags interval;
-nft__nat_postrouting_rules: {}
+  out_tcp_accept:
-                                                                   # ]]]
+    - type inet_service; flags interval;
-# .. envvar:: nft__nat_group_postrouting_rules [[[
+    - elements = $out_tcp_accept
-#
+  out_udp_accept:
-# List of postrouting rules to configure for hosts in specific
+    - type inet_service; flags interval;
-# Ansible inventory group.
+    - elements = $out_udp_accept
-nft__nat_group_postrouting_rules: {}
+nft_set_group: {}
-                                                                   # ]]]
+nft_set_host: {}
 # .. envvar:: nft__nat_host_postrouting_rules [[[
 #
 # List of postrouting rules to configure for specific hosts
 # in the Ansible inventory.
 nft__nat_host_postrouting_rules: {}
                                                                   # ]]]
 # .. envvar:: nft__nat_postrouting_conf_path [[[
 #
 # Path to the postrouting rules file for the nat table to include in the main
 # configuration file in order to use the previous defined lists.
 #
 # Should include the '{{ nft_conf_dir_path }}' var or be an absolut path.
 nft__nat_postrouting_conf_path: '{{ nft_conf_dir_path }}/nat-postrouting.nft'
                                                                   # ]]]
 # .. envvar:: nft__nat_postrouting_conf_content [[[
 #
 # Template used to provide the previous postrouting rules file.
 #
 # Must be a relative path from default/ directory of this role or from your
 # Ansible inventory directory.
 nft__nat_postrouting_conf_content: 'etc/nftables.d/nat-postrouting.nft.j2'
                                                                   # ]]]
                                                                   # ]]]
 # Service management [[[
 # ----------------------
-# .. envvar:: nft_service_manage [[[
+# service
 #
 # If the Nftables service should be managed ? Possible options are :
 #
 # ``True``
 #   Default. The service is started.
 #
 # ``False``
 #   The service will not be touched.
 nft_service_manage: true
                                                                   # ]]]
 # .. envvar:: nft_service_name [[[
 #
 # The service name to manage.
 nft_service_name: 'nftables'
                                                                   # ]]]
 # .. envvar:: nft_service_enabled [[[
 #
 # If the Nftables service should be enabled at startup ? Possible options are :
 #
 # ``True``
 #   Default. The service is enabled.
 #
 # ``False``
 #   The service is disabled from startup.
 nft_service_enabled: true
                                                                   # ]]]
 # .. envvar:: nft_service_unit_path [[[
 #
 # Path to store Nftables service.
 nft_service_unit_path: '/lib/systemd/system/nftables.service'
                                                                   # ]]]
 # .. envvar:: nft_service_unit_content [[[
 #
 # Template used to provide systemd unit for Nftables service.
 nft_service_unit_content: 'lib/systemd/system/nftables.service.j2'
                                                                   # ]]]
 # .. envvar:: nft__service_override_path [[[
 #
 # Path to store Nftables custom conf.
 nft__service_override_path: '/etc/systemd/system/nftables.service.d/override.conf'
                                                                   # ]]]
 # .. envvar:: nft__service_override_content [[[
 #
 # Template used to provide systemd custom conf for Nftables service.
 nft__service_override_content: 'etc/systemd/system/nftables.service.d/override.conf.j2'
                                                                   # ]]]
 # .. envvar:: nft__service_protect [[[
 #
 # If the systemd unit should have the Protect directives ? Possible options :
 #
 # ``True``
 #   Default. Directives will be set (ProtectSystem, ProtectHome,…).
 #
 # ``False``
 #   The directives will be ignored.
 nft__service_protect: true
                                                                   # ]]]
 # .. envvar:: nft__fail2ban_service_unit_path [[[
 #
 # Path to store Fail2Ban custom conf.
 nft__fail2ban_service_unit_path: '/etc/systemd/system/fail2ban.service.d/override.conf'
                                                                   # ]]]
 # .. envvar:: nft__fail2ban_service_unit_content [[[
 #
 # Template used to provide systemd custom conf for Fail2Ban service.
 nft__fail2ban_service_unit_content: 'etc/systemd/system/fail2ban.service.d/override.conf.j2'
                                                                   # ]]]
                                                                   #
 # .. envvar:: nft_debug [[[
 #
 # Toggle on/off more verbose output. Possible options are:
 #
 # ''Flase''
 # Default. No additional output will be given.
 #
 # ''True''
 # More verbose output.
 nft_debug: False
 # .. envvar:: nft_backup_conf [[[
 #
 # If the nftables config files should be backuped when changed ?
 # Possible options are:
 #
 # ``True``
 #   Default. Backup all nftables config files inside the nftables directory.
 #
 # ``False``
 #   Configs will not be backuped.
 nft_backup_conf: True
                                                                   # ]]]
                                                                   # ]]]
                                                                   # ]]]
 # OS specific variables defaults [[[
 # ----------------------------------
 # .. envvar:: nft__bin_location [[[
 #
 # Specify Nftables executable location.
 #
 nft__bin_location: '/usr/sbin/nft'
                                                                   # ]]]
                                                                   # ]]]
--- a/handlers/main.yml
+++ b/handlers/main.yml
@ -1,23 +1,5 @@
 ---
 # handlers file for nftables
-
+- name: restart nftables service
-# (re)Start will be called at first run
+  service: name={{ nft_service_name }} state=restarted
- name: Restart nftables service
+  when: nft_service_manage
  systemd:
    daemon_reload: '{{ (nftables__register_systemd_service.changed | default(False)) or
                       (nftables__register_fail2ban_service.changed | default(False)) or
                       (nftables__register_fix_systemd_target.changed | default(False)) or
                       (nftables__register_systemd_custom.changed | default(False)) }}'
    state: 'restarted'
    name: '{{ nft_service_name }}'
    enabled: '{{ nft_service_enabled }}'
  when: ansible_service_mgr == 'systemd' and nft_service_manage
 # Reload will avoid to loose Nftables rulebase if an invalid syntax is added
 - name: Reload nftables service
  systemd:
    state: 'reloaded'
    name: '{{ nft_service_name }}'
  when: ansible_service_mgr == 'systemd' and
        nft_service_manage and
        not nftables__register_systemd_service.changed
--- a/meta/main.yml
+++ b/meta/main.yml
@ -1,34 +1,22 @@
 ---
 dependencies: []
 galaxy_info:
  author: "Jérémy Gardais"
  namespace: ipr-cnrs
  role_name: nftables
  description: "Manage Nftables rules and packages"
  license: WTFPL
  company: IPR
-  issue_tracker_url: https://git.ipr.univ-rennes.fr/cellinfo/ansible.nftables/issues
+  issue_tracker_url: https://git.ipr.univ-rennes1.fr/cellinfo/ansible.nftables/issues
-  min_ansible_version: '2.5'
+  min_ansible_version: 2.2
  platforms:
  - name: Debian
    versions:
    - bullseye
    - buster
    - stretch
-  - name: Archlinux
+  #- name: opensuse
-    versions:
+  #  versions:
-    - all
+  #  - all
-  - name: Fedora
+  #  - 12.1
-    versions:
+  #  - 12.2
-    - all
+  #  - 12.3
-  - name: Gentoo
+  #  - 13.1
-    versions:
+  #  - 13.2
    - all
  - name: Ubuntu
    versions:
    - focal
  galaxy_tags:
    - system
    - nftables
--- a/molecule/archlinux/Dockerfile.j2
+++ b/molecule/archlinux/Dockerfile.j2
@ -1,7 +0,0 @@
 FROM archlinux:latest
 ENV container=docker
 RUN pacman -Sy --noconfirm python
 VOLUME ["/sys/fs/cgroup", "/tmp", "/run"]
 CMD ["/usr/sbin/init"]
--- a/molecule/archlinux/converge.yml
+++ b/molecule/archlinux/converge.yml
@ -1,9 +0,0 @@
 ---
 - name: Converge
  hosts: all
  gather_facts: yes
  roles:
    - role: ipr-cnrs.nftables
      nft_debug: true
      # can't remove iptables on an instance with docker
      nft_old_pkg_manage: false
--- a/molecule/archlinux/molecule.yml
+++ b/molecule/archlinux/molecule.yml
@ -1,19 +0,0 @@
 ---
 dependency:
  name: galaxy
 driver:
  name: docker
 platforms:
  - name: archlinux
    image: archlinux:latest
    command: /usr/sbin/init
    privileged: true
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:ro
    tmpfs:
      - /run
      - /tmp
 provisioner:
  name: ansible
 verifier:
  name: ansible
--- a/molecule/archlinux/verify.yml
+++ b/molecule/archlinux/verify.yml
@ -1,92 +0,0 @@
 ---
 # This is an example playbook to execute Ansible tests.
 - name: Verify
  hosts: all
  gather_facts: false
  tasks:
  - name: check for nftables.d
    stat:
      path: /etc/nftables.d
    register: p
  - name: check nftables.d
    assert:
      that:
        - p.stat.exists and p.stat.isdir
  - name: check for nftables.conf
    stat:
      path: /etc/nftables.conf
    register: p
  - name: check nftables.conf
    assert:
      that:
        - p.stat.exists
  - name: check for filter-input.nft
    stat:
      path: /etc/nftables.d/filter-input.nft
    register: p
  - name: check filter-input.nft
    assert:
      that:
        - p.stat.exists
  - name: list rules
    command: nft list ruleset
    register: nft
  - name: debug rules
    debug: var=nft
  - name: check rules
    assert:
      that:
        # The whole line is:
        # type filter hook input priority 0; policy drop;
        # However on CentOS will return "priority 0", while Debian will
        # show "priority filter"
        - '"type filter hook input" in nft.stdout'
        - '"type filter hook output" in nft.stdout'
  - name: check for fail2ban systemd custom dir
    stat:
      path: /etc/systemd/system/fail2ban.service.d
    register: f2b_systemd_dir
  - name: check fail2ban systemd custom dir
    assert:
      that:
        - f2b_systemd_dir.stat.exists and f2b_systemd_dir.stat.isdir
  - name: check for fail2ban systemd override
    stat:
      path: /etc/systemd/system/fail2ban.service.d/override.conf
    register: f2b_systemd_override
  - name: check fail2ban systemd override
    assert:
      that:
        - f2b_systemd_override.stat.exists
  - name: service status - active
    command: systemctl is-active nftables.service
    register: status
  - name: check service status
    assert:
      that:
        - 'status.stdout == "active"'
  - name: service status - enabled
    command: systemctl is-enabled nftables.service
    register: status
  - name: check service status
    assert:
      that:
        - 'status.stdout == "enabled"'
--- a/molecule/default/converge.yml
+++ b/molecule/default/converge.yml
@ -1,9 +0,0 @@
 ---
 - name: Converge
  hosts: all
  gather_facts: yes
  roles:
    - role: ipr-cnrs.nftables
      nft_debug: true
      # can't remove iptables on an instance with docker
      nft_old_pkg_manage: false
--- a/molecule/default/molecule.yml
+++ b/molecule/default/molecule.yml
@ -1,55 +0,0 @@
 ---
 dependency:
  name: galaxy
 lint: |
  set -e
  yamllint .
  ansible-lint
 driver:
  name: docker
 platforms:
  - name: systemd-ubuntu-latest
    image: jrei/systemd-ubuntu:latest
    command: /usr/sbin/init
    privileged: true
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:ro
    tmpfs:
      - /run
      - /tmp
  - name: systemd-centos-latest
    image: centos/systemd:latest
    command: /usr/sbin/init
    privileged: true
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:ro
    tmpfs:
      - /run
      - /tmp
  - name: systemd-debian-latest
    image: jrei/systemd-debian:latest
    command: /sbin/init
    privileged: true
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:ro
    tmpfs:
      - /run
      - /tmp
  - name: systemd-fedora-latest
    image: jrei/systemd-fedora:latest
    command: /usr/sbin/init
    privileged: true
    volumes:
      - /sys/fs/cgroup:/sys/fs/cgroup:ro
    tmpfs:
      - /run
      - /tmp
 provisioner:
  name: ansible
 verifier:
  name: ansible
--- a/molecule/default/verify.yml
+++ b/molecule/default/verify.yml
@ -1,92 +0,0 @@
 ---
 # This is an example playbook to execute Ansible tests.
 - name: Verify
  hosts: all
  gather_facts: false
  tasks:
  - name: check for nftables.d
    stat:
      path: /etc/nftables.d
    register: p
  - name: check nftables.d
    assert:
      that:
        - p.stat.exists and p.stat.isdir
  - name: check for nftables.conf
    stat:
      path: /etc/nftables.conf
    register: p
  - name: check nftables.conf
    assert:
      that:
        - p.stat.exists
  - name: check for filter-input.nft
    stat:
      path: /etc/nftables.d/filter-input.nft
    register: p
  - name: check filter-input.nft
    assert:
      that:
        - p.stat.exists
  - name: list rules
    command: nft list ruleset
    register: nft
  - name: debug rules
    debug: var=nft
  - name: check rules
    assert:
      that:
        # The whole line is:
        # type filter hook input priority 0; policy drop;
        # However on CentOS will return "priority 0", while Debian will
        # show "priority filter"
        - '"type filter hook input" in nft.stdout'
        - '"type filter hook output" in nft.stdout'
  - name: check for fail2ban systemd custom dir
    stat:
      path: /etc/systemd/system/fail2ban.service.d
    register: f2b_systemd_dir
  - name: check fail2ban systemd custom dir
    assert:
      that:
        - f2b_systemd_dir.stat.exists and f2b_systemd_dir.stat.isdir
  - name: check for fail2ban systemd override
    stat:
      path: /etc/systemd/system/fail2ban.service.d/override.conf
    register: f2b_systemd_override
  - name: check fail2ban systemd override
    assert:
      that:
        - f2b_systemd_override.stat.exists
  - name: service status - active
    command: systemctl is-active nftables.service
    register: status
  - name: check service status
    assert:
      that:
        - 'status.stdout == "active"'
  - name: service status - enabled
    command: systemctl is-enabled nftables.service
    register: status
  - name: check service status
    assert:
      that:
        - 'status.stdout == "enabled"'
--- a/molecule/gentoo/Dockerfile.j2
+++ b/molecule/gentoo/Dockerfile.j2
@ -1,5 +0,0 @@
 FROM gentoo/stage3:systemd
 ENV container=docker
 VOLUME ["/sys/fs/cgroup"]
 CMD ["/sbin/init"]
--- a/molecule/gentoo/converge.yml
+++ b/molecule/gentoo/converge.yml
@ -1,9 +0,0 @@
 ---
 - name: Converge
  hosts: all
  gather_facts: yes
  roles:
    - role: ipr-cnrs.nftables
      nft_debug: true
      # can't remove iptables on an instance with docker
      nft_old_pkg_manage: false
--- a/molecule/gentoo/molecule.yml
+++ b/molecule/gentoo/molecule.yml
@ -1,21 +0,0 @@
 ---
 dependency:
  name: galaxy
 driver:
  name: docker
 platforms:
  - name: Gentoo
    image: gentoo/stage3:systemd
    command: /sbin/init
    privileged: true
    volumes:
      - /srv/gentoo-molecule/gentoo-repo:/var/db/repos/gentoo
      - /srv/gentoo-molecule/binpkgs:/var/cache/binpkgs
      - /sys/fs/cgroup:/sys/fs/cgroup:ro
    tmpfs:
      - /run
      - /tmp
 provisioner:
  name: ansible
 verifier:
  name: ansible
--- a/molecule/gentoo/prepare.yml
+++ b/molecule/gentoo/prepare.yml
@ -1,34 +0,0 @@
 # Taken from https://github.com/VTimofeenko/portage-overlay-cfg on commit d8914035e236c4f3819985098dd1ae36551bfc52
 # If bugs are found – check that repository
 # Since Gentoo builds from source, to save time on molecule tests, we should reuse artifacts as much as possible
 # This playbook performs the initial setup of a Gentoo container
 # It configures portage to try to use prebuilt packages if available and to save the built packages.
 ---
 - name: Run preparation playbook
  hosts: Gentoo
  tasks:
    - name: Enable buildpkg feature
      lineinfile:
        line: "FEATURES='buildpkg'"
        dest: /etc/portage/make.conf
        state: present
    - name: Enable trying to install from binpkgs by default
      lineinfile:
        line: "EMERGE_DEFAULT_OPTS='--usepkg'"
        dest: /etc/portage/make.conf
        state: present
    - name: Synchronize gentoo repository if needed
      block:
        - name: Check if there is anything in the repo
          find:
            paths: '/var/db/repos/gentoo/'
          register: find_files_in_repo
        - name: Synchronize the repo if needed
          command: "emaint sync -a"
          when: find_files_in_repo.matched == 0
    - name: Install equery, needed for package check
      # Command, because equery does not exist yet
      command: emerge --changed-use --oneshot app-portage/gentoolkit
      changed_when: false
 # The rest of the original file is specific to that role and not needed here
--- a/molecule/gentoo/verify.yml
+++ b/molecule/gentoo/verify.yml
@ -1,92 +0,0 @@
 ---
 # This is an example playbook to execute Ansible tests.
 - name: Verify
  hosts: all
  gather_facts: false
  tasks:
  - name: check for nftables.d
    stat:
      path: /etc/nftables.d
    register: p
  - name: check nftables.d
    assert:
      that:
        - p.stat.exists and p.stat.isdir
  - name: check for nftables.conf
    stat:
      path: /etc/nftables.conf
    register: p
  - name: check nftables.conf
    assert:
      that:
        - p.stat.exists
  - name: check for filter-input.nft
    stat:
      path: /etc/nftables.d/filter-input.nft
    register: p
  - name: check filter-input.nft
    assert:
      that:
        - p.stat.exists
  - name: list rules
    command: nft list ruleset
    register: nft
  - name: debug rules
    debug: var=nft
  - name: check rules
    assert:
      that:
        # The whole line is:
        # type filter hook input priority 0; policy drop;
        # However on CentOS will return "priority 0", while Debian will
        # show "priority filter"
        - '"type filter hook input" in nft.stdout'
        - '"type filter hook output" in nft.stdout'
  - name: check for fail2ban systemd custom dir
    stat:
      path: /etc/systemd/system/fail2ban.service.d
    register: f2b_systemd_dir
  - name: check fail2ban systemd custom dir
    assert:
      that:
        - f2b_systemd_dir.stat.exists and f2b_systemd_dir.stat.isdir
  - name: check for fail2ban systemd override
    stat:
      path: /etc/systemd/system/fail2ban.service.d/override.conf
    register: f2b_systemd_override
  - name: check fail2ban systemd override
    assert:
      that:
        - f2b_systemd_override.stat.exists
  - name: service status - active
    command: systemctl is-active nftables.service
    register: status
  - name: check service status
    assert:
      that:
        - 'status.stdout == "active"'
  - name: service status - enabled
    command: systemctl is-enabled nftables.service
    register: status
  - name: check service status
    assert:
      that:
        - 'status.stdout == "enabled"'
--- a/tasks/main.yml
+++ b/tasks/main.yml
@ -1,90 +1,37 @@
 ---
 # .. vim: foldmarker=[[[,]]]:foldmethod=marker
 #
 # tasks file for nftables
- name: Check for group files
+- name: Load specific OS vars for nft
-  become: no
+  include_vars: "{{ item }}"
  delegate_to: localhost
  stat:
    path: "{{ nft_merged_groups_dir ~ groupname }}"
  register: nftables_group_rules
  loop: "{{ group_names }}"
  loop_control:
    loop_var: groupname
 - name: Debug nftables_group_rules
  debug: var=nftables_group_rules
  when: nft_debug
 - name: Import nftables-variables if nft_merged_groups is set
  when: nft_merged_groups and varfile.stat.exists
  include_vars:
    file: "{{ nft_merged_groups_dir ~ varfile.groupname }}"
    name: "{{ varfile.groupname }}"
  loop: "{{ nftables_group_rules.results }}"
  loop_control:
    loop_var: varfile
 - name: Combine Rules when nft_merged_groups is set
  when: nft_merged_groups and
        ((hostvars[inventory_hostname][varfile.groupname] is defined) and
         (hostvars[inventory_hostname][varfile.groupname]|length > 0)) and
        varfile.stat.exists
  set_fact:
    nft_combined_rules: "{{ nft_combined_rules | default({}) | combine ( hostvars[inventory_hostname][varfile.groupname], recursive=True ) }}"
  loop: "{{ nftables_group_rules.results }}"
  loop_control:
    loop_var: varfile
 - name: Debug nft_combined_rules
  debug: var=nft_combined_rules
  when: nft_debug
 - name: Debug ansible_os_family
  debug: var=ansible_os_family
  when: nft_debug
 - name: Load specific OS vars for nftables
  include_vars: "{{ osname }}"
  with_first_found:
    - "{{ ansible_distribution|lower }}-{{ ansible_distribution_version }}.yml"
    - "{{ ansible_distribution|lower }}.yml"
    - "{{ ansible_os_family|lower }}.yml"
  loop_control:
    loop_var: osname
-# Manage packages [[[1
+# package {{{
- name: Update repositories
+- name: INSTALL Manage nftables packages
  package:
-    update_cache: true
+    name: '{{ item }}'
  when: (nft_enabled|bool and
         ansible_os_family not in [ 'Gentoo' ])
 - name: Ensure Nftables packages are in their desired state
  package:
    name: '{{ nft_pkg_list | list }}'
    state: '{{ nft_pkg_state }}'
-  register: pkg_install_result
+  with_items:
-  until: pkg_install_result is success
+    - '{{ nft_pkg_list }}'
-  when: nft_enabled|bool
+  when: nft_pkg_manage
- name: Ensure old Iptables packages are in their desired state
+- name: INSTALL Remove iptables packages
-  package:
+  apt:
-    name: '{{ nft_old_pkg_list | list }}'
+    name: '{{ item }}'
    state: '{{ nft_old_pkg_state }}'
-  register: pkg_remove_result
+  with_items:
-  until: pkg_remove_result is success
+    - '{{ nft_old_pkg_list }}'
-  when: (nft_enabled|bool and
+  when: nft_old_pkg_manage
-         nft_old_pkg_manage|bool)
+# }}}
-# Common configuration [[[1
+# conf {{{
- name: Ensure to create nftables.d directory
+- name: CONFIG create nftables.d dir
  file:
    path: "{{ nft_conf_dir_path }}"
    state: directory
    mode: 0755
  when: nft_enabled|bool
 - name: CONFIG generate main conf file
  template:
@ -93,9 +40,28 @@
    owner: root
    group: root
    mode: 0755
-    backup: "{{ nft_backup_conf }}"
+    backup: yes
-  notify: ['Reload nftables service']
+  notify: ['restart nftables service']
-  when: nft_enabled|bool
+
 - name: CONFIG generate input rules file
  template:
    src: "{{ nft_input_conf_content }}"
    dest: "{{ nft_input_conf_path }}"
    owner: root
    group: root
    mode: 0755
    backup: yes
  notify: ['restart nftables service']
 - name: CONFIG generate output rules file
  template:
    src: "{{ nft_output_conf_content }}"
    dest: "{{ nft_output_conf_path }}"
    owner: root
    group: root
    mode: 0755
    backup: yes
  notify: ['restart nftables service']
 - name: CONFIG generate vars definition file
  template:
@ -104,83 +70,23 @@
    owner: root
    group: root
    mode: 0755
-    backup: "{{ nft_backup_conf }}"
+    backup: yes
-  notify: ['Reload nftables service']
+  notify: ['restart nftables service']
  when: nft_enabled|bool
- name: CONFIG generate sets file
+- name: CONFIG generate sets and maps file
  template:
    src: "{{ nft_set_conf_content }}"
    dest: "{{ nft_set_conf_path }}"
    owner: root
    group: root
    mode: 0755
    backup: "{{ nft_backup_conf }}"
  notify: ['Reload nftables service']
  when: nft_enabled|bool
 # Filter table content [[[1
 - name: Filter table - generate input rules file
  template:
    src: "{{ nft_input_conf_content }}"
    dest: "{{ nft_input_conf_path }}"
    owner: root
    group: root
    mode: 0755
    backup: "{{ nft_backup_conf }}"
  notify: ['Reload nftables service']
  when: nft_enabled|bool
 - name: Filter table - generate output rules file
  template:
    src: "{{ nft_output_conf_content }}"
    dest: "{{ nft_output_conf_path }}"
    owner: root
    group: root
    mode: 0755
    backup: "{{ nft_backup_conf }}"
  notify: ['Reload nftables service']
  when: nft_enabled|bool
 - name: Filter table - generate forward rules file
  template:
    src: "{{ nft_forward_conf_content }}"
    dest: "{{ nft_forward_conf_path }}"
    owner: root
    group: root
    mode: 0755
    backup: yes
-  notify: ['Reload nftables service']
+  notify: ['restart nftables service']
-  when: (nft_enabled|bool and
+# }}}
         nft__forward_table_manage|bool)
-# Nat table content [[[1
+# service {{{
 - name: Nat table - generate prerouting rules file
  template:
    src: "{{ nft__nat_prerouting_conf_content }}"
    dest: "{{ nft__nat_prerouting_conf_path }}"
    owner: root
    group: root
    mode: 0755
    backup: "{{ nft_backup_conf }}"
  notify: ['Reload nftables service']
  when: (nft_enabled|bool and
         nft__nat_table_manage|bool)
- name: Nat table - generate postrouting rules file
+- name: install Debian systemd service unit
  template:
    src: "{{ nft__nat_postrouting_conf_content }}"
    dest: "{{ nft__nat_postrouting_conf_path }}"
    owner: root
    group: root
    mode: 0755
    backup: "{{ nft_backup_conf }}"
  notify: ['Reload nftables service']
  when: (nft_enabled|bool and
         nft__nat_table_manage|bool)
 # Manage nftables service [[[1
 - name: Install nftables Debian systemd service unit
  template:
    src: '{{ nft_service_unit_content }}'
    dest: '{{ nft_service_unit_path }}'
@ -188,63 +94,19 @@
    group: 'root'
    mode: '0644'
  register: nftables__register_systemd_service
-  when: (nft_enabled|bool and
+  when: nft_service_manage
-         nft_service_manage|bool)
+  notify: ['restart nftables service']
  notify: ['Restart nftables service']
- name: Ensure to remove nftables systemd service from old target
+- name: Reload systemd daemons
-  file:
+  command: systemctl daemon-reload
-    path: '/etc/systemd/system/multi-user.target.wants/nftables.service'
+  notify: ['restart nftables service']
-    state: absent
+  when: (nft_service_manage and
-  register: nftables__register_fix_systemd_target
+         nftables__register_systemd_service|changed)
  when: (nft_enabled|bool and
         nft_service_manage|bool)
  notify: ['Restart nftables service']
-# Manage custom nftables service [[[1
+- name: SERVICE manage '{{ nft_service_name }}'
- name: Create Nftables custom directory for systemd service
+  service:
-  file:
+    name: '{{ nft_service_name }}'
-    path: "{{ nft__service_override_path | dirname }}"
+    state: started
-    state: directory
+    enabled: '{{ nft_service_enabled }}'
-    mode: '0755'
+  when: nft_service_manage
-  when:
+# }}}
    - nft_enabled|bool
    - nft_service_manage|bool
    - not nft__service_protect|bool
 - name: Add Nftables systemd custom configuration
  template:
    src: '{{ nft__service_override_content }}'
    dest: '{{ nft__service_override_path }}'
    owner: 'root'
    group: 'root'
    mode: '0644'
  register: nftables__register_systemd_custom
  when:
    - nft_enabled|bool
    - nft_service_manage|bool
    - not nft__service_protect|bool
  notify: ['Restart nftables service']
 # Manage custom fail2ban service [[[1
 - name: Create Fail2Ban custom directory for systemd service
  file:
    path: "{{ nft__fail2ban_service_unit_path | dirname }}"
    state: directory
    mode: '0755'
  when:
    - nft_enabled|bool
    - nft_service_manage|bool
 - name: Install Debian Fail2Ban custom service
  template:
    src: '{{ nft__fail2ban_service_unit_content }}'
    dest: '{{ nft__fail2ban_service_unit_path }}'
    owner: 'root'
    group: 'root'
    mode: '0644'
  register: nftables__register_fail2ban_service
  when:
    - nft_enabled|bool
    - nft_service_manage|bool
  notify: ['Restart nftables service']
--- a/templates/etc/nftables.conf.j2
+++ b/templates/etc/nftables.conf.j2
@ -1,18 +1,13 @@
-#jinja2: lstrip_blocks: "True", trim_blocks: "True"
+#!/usr/sbin/nft -f
 #!{{ nft__bin_location }} -f
 # {{ ansible_managed }}
 {% set globalmerged = nft_global_default_rules.copy() %}
 {% set _ = globalmerged.update(nft_global_rules) %}
 {% set _ = globalmerged.update(nft_global_group_rules) %}
 {% if nft_merged_groups and hostvars[inventory_hostname]['nft_combined_rules'].nft_global_group_rules is defined%}
  {% set _ = globalmerged.update(hostvars[inventory_hostname]['nft_combined_rules'].nft_global_group_rules) %}
 {% endif %}
 {% set _ = globalmerged.update(nft_global_host_rules) %}
 # clean
 flush ruleset
-include "{{ nft_define_conf_path }}"
+include "/etc/nftables.d/defines.nft"
 table inet filter {
 	chain global {
@ -29,30 +24,4 @@ table inet filter {
 	include "{{ nft_set_conf_path }}"
 	include "{{ nft_input_conf_path }}"
 	include "{{ nft_output_conf_path }}"
 {% if nft__forward_table_manage %}
 	include "{{ nft_forward_conf_path }}"
 {% endif %}
 {% if nft_custom_includes | default() %}
  {% if nft_custom_includes is string %}
 	include "{{ nft_custom_includes }}"
  {% elif nft_custom_includes is iterable and (nft_custom_includes is not string and nft_custom_includes is not mapping) %}
    {% for include in nft_custom_includes %}
 	include "{{ include }}"
    {% endfor %}
  {% endif %}
 {% endif %}
 }
 {% if nft__nat_table_manage %}
 # Additionnal table for Network Address Translation (NAT)
 table ip nat {
 	include "{{ nft_set_conf_path }}"
 	include "{{ nft__nat_prerouting_conf_path }}"
 	include "{{ nft__nat_postrouting_conf_path }}"
 }
 {% endif %}
 {% if nft__custom_content|d() %}
 # Custom content from ipr-cnrs.nftables
 {{ nft__custom_content }}
 {% endif %}
--- a/templates/etc/nftables.d/defines.nft.j2
+++ b/templates/etc/nftables.d/defines.nft.j2
@ -1,13 +1,9 @@
 #jinja2: lstrip_blocks: "True", trim_blocks: "True"
 # {{ ansible_managed }}
 {% set definemerged = nft_define_default.copy() %}
 {% set _ = definemerged.update(nft_define) %}
 {% set _ = definemerged.update(nft_define_group) %}
 {% if nft_merged_groups and hostvars[inventory_hostname]['nft_combined_rules'].nft_define_group is defined%}
  {% set _ = definemerged.update(hostvars[inventory_hostname]['nft_combined_rules'].nft_define_group) %}
 {% endif %}
 {% set _ = definemerged.update(nft_define_host) %}
 {% for definition in definemerged.values() %}
 {% if definition.desc is defined %}
 # {{ definition.desc }}
--- a/templates/etc/nftables.d/filter-forward.nft.j2
+++ b/templates/etc/nftables.d/filter-forward.nft.j2
@ -1,21 +0,0 @@
 #jinja2: lstrip_blocks: "True", trim_blocks: "True"
 # {{ ansible_managed }}
 {% set forwardmerged = nft_forward_default_rules.copy() %}
 {% set _ = forwardmerged.update(nft_forward_rules) %}
 {% set _ = forwardmerged.update(nft_forward_group_rules) %}
 {% if nft_merged_groups and hostvars[inventory_hostname]['nft_combined_rules'].nft_forward_group_rules is defined %}
  {% set _ = forwardmerged.update(hostvars[inventory_hostname]['nft_combined_rules'].nft_forward_group_rules) %}
 {% endif %}
 {% set _ = forwardmerged.update(nft_forward_host_rules) %}
 chain forward {
 {% for group, rules in forwardmerged|dictsort  %}
    # {{ group }}
  {% if not rules %}
    # (none)
  {% endif %}
  {% for rule in rules %}
    {{ rule }}
  {% endfor %}
 {% endfor %}
 }
--- a/templates/etc/nftables.d/filter-input.nft.j2
+++ b/templates/etc/nftables.d/filter-input.nft.j2
@ -1,11 +1,6 @@
 #jinja2: lstrip_blocks: "True", trim_blocks: "True"
 # {{ ansible_managed }}
 {% set inputmerged = nft_input_default_rules.copy() %}
 {% set _ = inputmerged.update(nft_input_rules) %}
 {% set _ = inputmerged.update(nft_input_group_rules) %}
 {% if nft_merged_groups and hostvars[inventory_hostname]['nft_combined_rules'].nft_input_group_rules is defined %}
  {% set _ = inputmerged.update(hostvars[inventory_hostname]['nft_combined_rules'].nft_input_group_rules) %}
 {% endif %}
 {% set _ = inputmerged.update(nft_input_host_rules) %}
 chain input {
--- a/templates/etc/nftables.d/filter-output.nft.j2
+++ b/templates/etc/nftables.d/filter-output.nft.j2
@ -1,11 +1,6 @@
 #jinja2: lstrip_blocks: "True", trim_blocks: "True"
 # {{ ansible_managed }}
 {% set outputmerged = nft_output_default_rules.copy() %}
 {% set _ = outputmerged.update(nft_output_rules) %}
 {% set _ = outputmerged.update(nft_output_group_rules) %}
 {% if nft_merged_groups and hostvars[inventory_hostname]['nft_combined_rules'].nft_output_group_rules is defined %}
  {% set _ = outputmerged.update(hostvars[inventory_hostname]['nft_combined_rules'].nft_output_group_rules) %}
 {% endif %}
 {% set _ = outputmerged.update(nft_output_host_rules) %}
 chain output {
--- a/templates/etc/nftables.d/nat-postrouting.nft.j2
+++ b/templates/etc/nftables.d/nat-postrouting.nft.j2
@ -1,21 +0,0 @@
 #jinja2: lstrip_blocks: "True", trim_blocks: "True"
 # {{ ansible_managed }}
 {% set postroutingmerged = nft__nat_default_postrouting_rules.copy() %}
 {% set _ = postroutingmerged.update(nft__nat_postrouting_rules) %}
 {% set _ = postroutingmerged.update(nft__nat_group_postrouting_rules) %}
 {% if nft_merged_groups and hostvars[inventory_hostname]['nft_combined_rules'].nft__nat_group_postrouting_rules is defined %}
  {% set _ = postroutingmerged.update(hostvars[inventory_hostname]['nft_combined_rules'].nft__nat_group_postrouting_rules) %}
 {% endif %}
 {% set _ = postroutingmerged.update(nft__nat_host_postrouting_rules) %}
 chain postrouting {
 {% for group, rules in postroutingmerged|dictsort  %}
 	# {{ group }}
  {% if not rules %}
 	# (none)
  {% endif %}
  {% for rule in rules %}
 	{{ rule }}
  {% endfor %}
 {% endfor %}
 }
--- a/templates/etc/nftables.d/nat-prerouting.nft.j2
+++ b/templates/etc/nftables.d/nat-prerouting.nft.j2
@ -1,21 +0,0 @@
 #jinja2: lstrip_blocks: "True", trim_blocks: "True"
 # {{ ansible_managed }}
 {% set preroutingmerged = nft__nat_default_prerouting_rules.copy() %}
 {% set _ = preroutingmerged.update(nft__nat_prerouting_rules) %}
 {% set _ = preroutingmerged.update(nft__nat_group_prerouting_rules) %}
 {% if nft_merged_groups and hostvars[inventory_hostname]['nft_combined_rules'].nft__nat_group_prerouting_rules is defined %}
  {% set _ = preroutingmerged.update(hostvars[inventory_hostname]['nft_combined_rules'].nft__nat_group_prerouting_rules) %}
 {% endif %}
 {% set _ = preroutingmerged.update(nft__nat_host_prerouting_rules) %}
 chain prerouting {
 {% for group, rules in preroutingmerged|dictsort  %}
 	# {{ group }}
  {% if not rules %}
 	# (none)
  {% endif %}
  {% for rule in rules %}
 	{{ rule }}
  {% endfor %}
 {% endfor %}
 }
--- a/templates/etc/nftables.d/sets.nft.j2
+++ b/templates/etc/nftables.d/sets.nft.j2
@ -1,11 +1,6 @@
 #jinja2: lstrip_blocks: "True", trim_blocks: "True"
 # {{ ansible_managed }}
 {% set setmerged = nft_set_default.copy() %}
 {% set _ = setmerged.update(nft_set) %}
 {% set _ = setmerged.update(nft_set_group) %}
 {% if nft_merged_groups and hostvars[inventory_hostname]['nft_combined_rules'].nft_set_group is defined %}
  {% set _ = setmerged.update(hostvars[inventory_hostname]['nft_combined_rules'].nft_set_group) %}
 {% endif %}
 {% set _ = setmerged.update(nft_set_host) %}
 {% for set, rules in setmerged|dictsort  %}
--- a/templates/etc/systemd/system/fail2ban.service.d/override.conf.j2
+++ b/templates/etc/systemd/system/fail2ban.service.d/override.conf.j2
@ -1,8 +0,0 @@
 # {{ ansible_managed }}
 [Unit]
 After=network.target iptables.service firewalld.service ip6tables.service ipset.service nftables.service
 PartOf=firewalld.service nftables.service
 [Install]
 WantedBy=multi-user.target nftables.service
--- a/templates/etc/systemd/system/nftables.service.d/override.conf.j2
+++ b/templates/etc/systemd/system/nftables.service.d/override.conf.j2
@ -1,7 +0,0 @@
 # {{ ansible_managed }}
 [Service]
 {% if not nft__service_protect %}
 ProtectSystem=no
 ProtectHome=no
 {% endif %}
--- a/templates/lib/systemd/system/nftables.service.j2
+++ b/templates/lib/systemd/system/nftables.service.j2
@ -2,10 +2,7 @@
 [Unit]
 Description={{ nft_service_name }}
 Documentation=man:nft(8) http://wiki.nftables.org
-Wants=network-pre.target
+Before=fail2ban.service
 Before=network-pre.target shutdown.target
 Conflicts=shutdown.target
 DefaultDependencies=no
 [Service]
 Type=oneshot
@ -13,9 +10,9 @@ RemainAfterExit=yes
 StandardInput=null
 ProtectSystem=full
 ProtectHome=true
-ExecStart={{ nft__bin_location }} -f {{ nft_main_conf_path }}
+ExecStart=/usr/sbin/nft -f {{ nft_main_conf_path }}
-ExecReload={{ nft__bin_location }} -f {{ nft_main_conf_path }}
+ExecReload=/usr/sbin/nft -f {{ nft_main_conf_path }}
-ExecStop={{ nft__bin_location }} flush ruleset
+ExecStop=/usr/sbin/nft flush ruleset
 [Install]
-WantedBy=sysinit.target
+WantedBy=multi-user.target
--- a/vars/alpine.yml
+++ b/vars/alpine.yml
@ -1,4 +0,0 @@
 ---
 # vars file for Alpine
 nft_pkg_list:
  - nftables
--- a/vars/archlinux.yml
+++ b/vars/archlinux.yml
@ -1,4 +0,0 @@
 ---
 # vars file for Archlinux-based distros
 nft_pkg_list:
  - nftables
--- a/vars/gentoo.yml
+++ b/vars/gentoo.yml
@ -1,5 +0,0 @@
 ---
 # vars file for Gentoo
 nft_pkg_list:
  - net-firewall/nftables
 nft__bin_location: "/sbin/nft"
--- a/vars/redhat.yml
+++ b/vars/redhat.yml
@ -1,4 +0,0 @@
 ---
 # vars file for Centos-based distros
 nft_pkg_list:
  - nftables