Merge pull request 'Leave the header checks to the list owners.' (!17) from adellam/mailman:master into master

Reviewed-on: ISTI-ansible-roles/ansible-role-mailman#17

README.md

@@ -1,2 +1,12 @@
 # mailman
 
+The site-wide antispam defaults are deliberately empty:
+
+```yaml
+mailman_antispam_header_checks: []
+mailman_antispam_chain_behaviour: discard
+```
+
+Mailing-list owners can manage list-specific header matches and select their
+action (`accept`, `discard`, `hold`, or `reject`) from Postorius. The global
+chain is used only when a matching rule has no explicit action.

defaults/main.yml

@@ -118,14 +118,14 @@ mailman_arc_enabled: 'no'
 mailman_dmarc_enabled: 'yes'
 mailman_dkim_enabled: 'yes'
 
-mailman_antispam_header_checks:
-  - 'X-Spam-Flag: (YES)'
-  #- 'Authentication-Results: mail.example.com; dmarc=(fail|quarantine)'
+# Site-wide checks are intentionally empty. List owners can configure header
+# matches, including their action, from Postorius for each individual list.
+mailman_antispam_header_checks: []
 
 # The chain to jump to if any of the header patterns matches.  This must be
 # the name of an existing chain such as 'discard', 'reject', 'hold', or
 # 'accept', otherwise 'hold' will be used.
-mailman_antispam_chain_behaviour: "hold"
+mailman_antispam_chain_behaviour: "discard"
 
 mailman_start_nntp_runner: 'no'
 
@@ -184,4 +184,3 @@ mailman_postorius_cronjob_special_times:
   - 'minutely'
 
 
-