aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMark Sapiro <mark@msapiro.net>2015-05-01 09:14:08 -0700
committerMark Sapiro <mark@msapiro.net>2015-05-01 09:14:08 -0700
commit47c050236d3632f0dfaca75b2c96d5c880355185 (patch)
treece7e96271f9eb20027da02eecdc53fd22d1faee9
parent8f7fc30bde024165d742ecae082858d80f1012b8 (diff)
downloadmailman2-47c050236d3632f0dfaca75b2c96d5c880355185.tar.gz
mailman2-47c050236d3632f0dfaca75b2c96d5c880355185.tar.xz
mailman2-47c050236d3632f0dfaca75b2c96d5c880355185.zip
The vette log entry for DMARC policy hits now contains the list name.
-rw-r--r--Mailman/Utils.py8
-rwxr-xr-xNEWS3
2 files changed, 7 insertions, 4 deletions
diff --git a/Mailman/Utils.py b/Mailman/Utils.py
index 1cd1cdb7..e7eaa389 100644
--- a/Mailman/Utils.py
+++ b/Mailman/Utils.py
@@ -1209,15 +1209,15 @@ def IsDMARCProhibited(mlist, email):
for entry in dmarcs:
if re.search(r'\bp=reject\b', entry, re.IGNORECASE):
syslog('vette',
- 'DMARC lookup for %s (%s) found p=reject in %s = %s',
- email, dmarc_domain, name, entry)
+ '%s: DMARC lookup for %s (%s) found p=reject in %s = %s',
+ mlist.real_name, email, dmarc_domain, name, entry)
return True
if (mlist.dmarc_quarantine_moderation_action and
re.search(r'\bp=quarantine\b', entry, re.IGNORECASE)):
syslog('vette',
- 'DMARC lookup for %s (%s) found p=quarantine in %s = %s',
- email, dmarc_domain, name, entry)
+ '%s: DMARC lookup for %s (%s) found p=quarantine in %s = %s',
+ mlist.real_name, email, dmarc_domain, name, entry)
return True
return False
diff --git a/NEWS b/NEWS
index e0c4951f..a91d936d 100755
--- a/NEWS
+++ b/NEWS
@@ -9,6 +9,9 @@ Here is a history of user visible changes to Mailman.
Bug fixes and other patches
+ - The vette log entry for DMARC policy hits now contains the list name.
+ (LP: #1450826)
+
- If SUBSCRIBE_FORM_SECRET is enabled and a user's network has a load
balancer or similar in use the POSTing IP might not exactly match the
GETting IP. This is now accounted for by not requiring the last