Mirror
/
app-MAIL-temp
mirror of https://github.com/simple-login/app.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #814 from acasajus/new/multiple-mx 

Allow to have lower priority MX servers that do not belong to simplelogin
This commit is contained in:
Son Nguyen Kim 2022-02-25 09:30:59 +01:00 committed by GitHub
parent 7da06ba424 0c008edc82
commit 4df32b3b03
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/dns_utils.py
View File

@ -112,10 +112,10 @@ def is_mx_equivalent(
ref_mx_domains, key=lambda priority_domain: priority_domain[0]
)
if len(mx_domains) != len(ref_mx_domains):
if len(mx_domains) < len(ref_mx_domains):
return False
for i in range(0, len(mx_domains)):
for i in range(0, len(ref_mx_domains)):
if mx_domains[i][1] != ref_mx_domains[i][1]:
return False

8
tests/test_dns_utils.py
View File

@ -38,3 +38,11 @@ def test_is_mx_equivalent():
assert is_mx_equivalent(
[(5, "domain1"), (10, "domain2")], [(10, "domain1"), (20, "domain2")]
)
assert is_mx_equivalent(
[(5, "domain1"), (10, "domain2"), (20, "domain3")],
[(10, "domain1"), (20, "domain2")],
)
assert not is_mx_equivalent(
[(5, "domain1"), (10, "domain2")],
[(10, "domain1"), (20, "domain2"), (20, "domain3")],
)