diff options
author | Mark Sapiro <mark@msapiro.net> | 2018-06-05 12:14:08 -0700 |
---|---|---|
committer | Mark Sapiro <mark@msapiro.net> | 2018-06-05 12:14:08 -0700 |
commit | 8291c814c54d87c7958304e471a5c5c013417e45 (patch) | |
tree | e4a3ff152cb9c352ecf42ccbee0db7c5f06de23c /Mailman | |
parent | ca506e913faa49331db68f541774fdb773653988 (diff) | |
download | mailman2-8291c814c54d87c7958304e471a5c5c013417e45.tar.gz mailman2-8291c814c54d87c7958304e471a5c5c013417e45.tar.xz mailman2-8291c814c54d87c7958304e471a5c5c013417e45.zip |
Extend BLOCK_SPAMHAUS_LISTED_IP_SUBSCRIBE for IPv6.
Fix string formatting for Python 2.6.
Diffstat (limited to 'Mailman')
-rw-r--r-- | Mailman/Utils.py | 28 |
1 files changed, 21 insertions, 7 deletions
diff --git a/Mailman/Utils.py b/Mailman/Utils.py index cdc82366..6038667b 100644 --- a/Mailman/Utils.py +++ b/Mailman/Utils.py @@ -79,6 +79,12 @@ try: except ImportError: dns_resolver = False +try: + import ipaddress + have_ipaddress = True +except ImportError: + have_ipaddress = False + EMPTYSTRING = '' UEMPTYSTRING = u'' CR = '\r' @@ -1498,13 +1504,21 @@ def xml_to_unicode(s, cset): def banned_ip(ip): if not dns_resolver: return False - parts = ip.split('.') - if len(parts) != 4: - return False - lookup = '{}.{}.{}.{}.zen.spamhaus.org'.format(parts[3], - parts[2], - parts[1], - parts[0]) + if have_ipaddress: + try: + uip = unicode(ip, encoding='us-ascii', errors='replace') + ptr = ipaddress.ip_address(uip).reverse_pointer + except ValueError: + return False + lookup = '{0}.zen.spamhaus.org'.format('.'.join(ptr.split('.')[:-2])) + else: + parts = ip.split('.') + if len(parts) != 4: + return False + lookup = '{0}.{1}.{2}.{3}.zen.spamhaus.org'.format(parts[3], + parts[2], + parts[1], + parts[0]) resolver = dns.resolver.Resolver() try: ans = resolver.query(lookup, dns.rdatatype.A) |