5.2.20

- Don't overcount lookups caused by the `mx` mechanism (Closes #197)

Author: seanthegeek

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 5.2.20
+
+- Don't overcount lookups caused by the `mx` mechanism (Closes #197)
+
 ## 5.2.19
 
 - Make warning text consistent

checkdmarc/_constants.py

@@ -19,7 +19,7 @@
 See the License for the specific language governing permissions and
 limitations under the License."""
 
-__version__ = "5.12.19"
+__version__ = "5.12.20"
 
 OS = platform.system()
 OS_RELEASE = platform.release()

checkdmarc/spf.py

@@ -454,16 +454,13 @@ def parse_spf_record(
                     )
 
                 # RFC 7208 § 4.6.4: no more than 10 DNS queries total per evaluation
-                if len(mx_hosts) > 9:
+                if len(mx_hosts) > 10:
                     raise SPFTooManyDNSLookups(
-                        f"{value} has more than 9 MX records - " "(RFC 7208 § 4.6.4)",
+                        f"{value} has more than 10 MX records - " "(RFC 7208 § 4.6.4)",
                         dns_lookups=len(mx_hosts),
                     )
                 host_ips = {}
                 for host in mx_hosts:
-                    # count one DNS query per MX target
-                    mechanism_dns_lookups += 1
-                    total_dns_lookups += 1
                     hostname = host["hostname"]
                     # --- perform A/AAAA resolution for each MX host ---
                     try: