From 05d5617c291f07cb3e969bde2f7d7621a8d5c33b Mon Sep 17 00:00:00 2001
From: ale <ale@incal.net>
Date: Mon, 30 Dec 2024 09:13:34 +0000
Subject: [PATCH 1/3] Enable VERP for (some) outbound emails

Improve bounce detection.

Also reduce the number of recipients per single SMTP delivery.
---
 roles/mailman/templates/mm_cfg.py.j2 | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/roles/mailman/templates/mm_cfg.py.j2 b/roles/mailman/templates/mm_cfg.py.j2
index 3fdbb8b8..b0cd8f89 100644
--- a/roles/mailman/templates/mm_cfg.py.j2
+++ b/roles/mailman/templates/mm_cfg.py.j2
@@ -113,7 +113,7 @@ DEFAULT_REPLY_GOES_TO_LIST = 0
 DEFAULT_MAX_DAYS_TO_HOLD = 15
 
 # rimuove le firme Dkim e Dk dalle mail in entrata a mailman
-REMOVE_DKIM_HEADERS = 'YES'
+REMOVE_DKIM_HEADERS = Yes
 
 
 #-------------------------------------------------------------
@@ -141,11 +141,14 @@ ARCHIVE_HTML_SANITIZER = 0
 # 3 = Discard
 DEFAULT_GENERIC_NONMEMBER_ACTION = 2
 
-# added by ale+blicero to limit recipient per send and avoid spam limit measures
-SMTP_MAX_RCPTS = 50
+# Enable VERP probes for better bounce detection, one in 10 emails.
+VERP_PROBES = Yes
+VERP_CONFIRMATIONS = Yes
+VERP_DELIVERY_INTERVAL = 10
 
-# Note - if you're looking for something that is imported from mm_cfg, but you
-# didn't find it above, it's probably in /usr/lib/mailman/Mailman/Defaults.py.
+# Limit recipient per send, avoid spam limit measures and limit blast
+# radius from delivery failures.
+SMTP_MAX_RCPTS = 10
 
 DEFAULT_SUBSCRIBE_POLICY = 3
 DEFAULT_LIST_ADVERTISED = No
-- 
GitLab


From abdbddde23286c3471f8d505e7a2fc5e3926af46 Mon Sep 17 00:00:00 2001
From: ale <ale@incal.net>
Date: Mon, 30 Dec 2024 09:58:04 +0000
Subject: [PATCH 2/3] Fix CI build

---
 .gitlab-ci.yml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 179095eb..7a0e2501 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -10,6 +10,8 @@ variables:
 docker_build_and_release_tests:
   stage: docker_build
   image: quay.io/podman/stable
+  variables:
+    IMAGE_TAG: $CI_REGISTRY_IMAGE:$CI_COMMIT_REF_SLUG
   script:
     - echo -n "$CI_JOB_TOKEN" | podman login -u gitlab-ci-token --password-stdin $CI_REGISTRY
     - cd test && podman build --build-arg ci_token=$CI_JOB_TOKEN --pull -t $IMAGE_TAG .
-- 
GitLab


From 21c06a78b4ac3bff367427cb141127c959da4c46 Mon Sep 17 00:00:00 2001
From: ale <ale@incal.net>
Date: Mon, 30 Dec 2024 12:57:28 +0000
Subject: [PATCH 3/3] Small fix

---
 roles/testdata-mailman/tasks/load.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/roles/testdata-mailman/tasks/load.yml b/roles/testdata-mailman/tasks/load.yml
index efc7957e..41e12b54 100644
--- a/roles/testdata-mailman/tasks/load.yml
+++ b/roles/testdata-mailman/tasks/load.yml
@@ -13,4 +13,4 @@
 
 - name: Subscribe test user to test list
   shell: "echo uno@investici.org | in-container mailman-mailman /var/lib/mailman/bin/add_members -w n -a n -r - {{ test_list_name }}"
-  when: "mailman_test_list.exists"
+  when: "mailman_test_list.exists | default(False)"
-- 
GitLab