do not mark a domain as unverified if it fails the MX check

This commit is contained in:
Son NK 2020-08-31 17:11:18 +02:00
parent 63e228d9f4
commit e4c4797cdb
2 changed files with 9 additions and 5 deletions

View File

@ -71,7 +71,10 @@
{% endfor %} {% endfor %}
</div> </div>
{% if custom_domain.verified %} {% if custom_domain.verified %}
Please make sure to fix this ASAP - your aliases might not work properly. <div class="alert alert-danger">
Without the MX record set up correctly, you can miss emails sent to your aliases.
Please update the MX record ASAP.
</div>
{% endif %} {% endif %}
</div> </div>
{% endif %} {% endif %}
@ -89,7 +92,8 @@
</div> </div>
<div> <div>
SPF <a href="https://en.wikipedia.org/wiki/Sender_Policy_Framework" target="_blank" rel="noopener">(Wikipedia↗)</a> is an email SPF <a href="https://en.wikipedia.org/wiki/Sender_Policy_Framework" target="_blank"
rel="noopener">(Wikipedia↗)</a> is an email
authentication method authentication method
designed to detect forging sender addresses during the delivery of the email. <br> designed to detect forging sender addresses during the delivery of the email. <br>
Setting up SPF is highly recommended to reduce the chance your emails ending up in the recipient's Spam folder. Setting up SPF is highly recommended to reduce the chance your emails ending up in the recipient's Spam folder.
@ -153,7 +157,8 @@
</div> </div>
<div> <div>
DKIM <a href="https://en.wikipedia.org/wiki/DomainKeys_Identified_Mail" target="_blank" rel="noopener">(Wikipedia↗)</a> is an DKIM <a href="https://en.wikipedia.org/wiki/DomainKeys_Identified_Mail" target="_blank" rel="noopener">(Wikipedia↗)</a>
is an
email email
authentication method authentication method
designed to avoid email spoofing. <br> designed to avoid email spoofing. <br>

View File

@ -37,8 +37,7 @@ def domain_detail_dns(custom_domain_id):
if sorted(mx_domains) != sorted(EMAIL_SERVERS_WITH_PRIORITY): if sorted(mx_domains) != sorted(EMAIL_SERVERS_WITH_PRIORITY):
flash("The MX record is not correctly set", "warning") flash("The MX record is not correctly set", "warning")
custom_domain.verified = False
db.session.commit()
mx_ok = False mx_ok = False
# build mx_errors to show to user # build mx_errors to show to user
mx_errors = [ mx_errors = [