[Bug 1287604] Re: Enhancement: Sanity Check Subscribers MX Record

Mark Sapiro mark at msapiro.net
Mon Feb 16 06:27:21 CET 2015


The major issue with your patch is not all valid email domains have MX
records. A valid email domain must have either an A record, an AAAA
record or an MX record pointing to a host name with an A or AAAA record.
E.g., a domain with an MX pointing to a host name with only a CNAME or
MX is not a valid email domain, but a domain with only an A or AAAA
record is a valid email domain. A domain which has a CNAME is a valid
email domain as long as the CNAME directly or indirectly points to a
host with an A, AAAA or MX as above. See RFC 5321, Sec 5.1.

Another issue is Mailman tries not to use bare except: clauses to catch
exceptions. We prefer to catch only those explicit exceptions which
might occur. I.e., in this case, explicitly only those exceptions that
might be returned by dns.resolver.query.

-- 
You received this bug notification because you are a member of Mailman
Coders, which is subscribed to GNU Mailman.
https://bugs.launchpad.net/bugs/1287604

Title:
  Enhancement: Sanity Check Subscribers MX Record

To manage notifications about this bug go to:
https://bugs.launchpad.net/mailman/+bug/1287604/+subscriptions


More information about the Mailman-coders mailing list