From 7e94adf84f3bdb0be75a209b06d81578a74e5439 Mon Sep 17 00:00:00 2001 From: Shane Frasier Date: Mon, 15 Jan 2018 16:26:28 -0500 Subject: [PATCH] The base domain name is already part of the Domain object, so I can just use that. --- trustymail/trustymail.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/trustymail/trustymail.py b/trustymail/trustymail.py index 03340fe..6def76a 100755 --- a/trustymail/trustymail.py +++ b/trustymail/trustymail.py @@ -474,10 +474,7 @@ def dmarc_scan(resolver, domain): domain.dmarc_forensic_uris.append(uri) email_address = parsed_uri["address"] email_domain = email_address.split('@')[-1] - # Domain.domain_name is set to lowercase in - # Donain's constructor, so there is no need to call - # lower() on the right-hand side of the comparison - if get_public_suffix(email_domain).lower() != domain.base_domain.domain_name: + if get_public_suffix(email_domain).lower() != domain.base_domain_name.lower(): target = '{0}._report._dmarc.{1}'.format(domain.domain_name, email_domain) error_message = '{0} does not indicate that it accepts DMARC reports about {1} - ' \ 'https://tools.ietf.org' \