DNSIMPLE: enable Null MX records (#2960)

StackExchange · May 16, 2024 · df1354b · df1354b
1 parent c7975f9
commit df1354b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/providers/dnsimple/auditrecords.go b/providers/dnsimple/auditrecords.go
@@ -11,8 +11,6 @@ import (
 func AuditRecords(records []*models.RecordConfig) []error {
 	a := rejectif.Auditor{}
 
-	a.Add("MX", rejectif.MxNull) // Last verified 2023-03
-
 	a.Add("TXT", rejectif.TxtLongerThan(1000)) // Last verified 2023-12
 
 	a.Add("TXT", rejectif.TxtHasTrailingSpace) // Last verified 2023-03

diff --git a/providers/dnsimple/dnsimpleProvider.go b/providers/dnsimple/dnsimpleProvider.go
@@ -95,7 +95,9 @@ func (c *dnsimpleProvider) GetZoneRecords(domain string, meta map[string]string)
 			r.Name = "@"
 		}
 
-		if r.Type == "CNAME" || r.Type == "MX" || r.Type == "ALIAS" || r.Type == "NS" {
+		if r.Type == "CNAME" || r.Type == "ALIAS" || r.Type == "NS" {
+			r.Content += "."
+		} else if r.Type == "MX" && r.Content != "." {
 			r.Content += "."
 		}