From: Ralf Jung <post@ralfj.de>
Date: Tue, 13 Nov 2018 09:47:57 +0000 (+0100)
Subject: improve mailman-check
X-Git-Url: https://git.ralfj.de/ansible.git/commitdiff_plain/f01791c935972bc8fa351f719d24ac4aee1d4eff?ds=sidebyside

improve mailman-check
---

diff --git a/roles/email/files/mailman-check b/roles/email/files/mailman-check
index 595b28c..0a5d9de 100755
--- a/roles/email/files/mailman-check
+++ b/roles/email/files/mailman-check
@@ -4,14 +4,8 @@ set -e
 ##   ./mailman-check
 ## Checks all the mailman lists for bad configuration, and prints further details if it finds one.
 
-if ! test -d /var/lib/mailman; then
-    # nothing to do
-    exit 0
-fi
-
-cd /var/lib/mailman/lists
-for list in *; do
-    /usr/sbin/config_list  -i /dev/stdin "$list" <<EOF
+for list in $(list_lists -b); do
+    /usr/sbin/config_list -c -i /dev/stdin "$list" <<EOF
 if not ((mlist.dmarc_moderation_action in (1, 2) and mlist.dmarc_quarantine_moderation_action == 1) or mlist.from_is_list in (1, 2)):
     print "List",mlist.real_name,"by",(', '.join(mlist.owner)),"is not configured to deal with DMARC."
 if mlist.reply_goes_to_list != 0 and not mlist.first_strip_reply_to: