fix(GoDaddy): use @ as name when subdomain is missing (#770)

jeessy2 · Jul 14, 2023 · e009a82 · e009a82
1 parent b66340e
commit e009a82
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/config/domains.go b/config/domains.go
@@ -45,7 +45,7 @@ func (d Domain) GetFullDomain() string {
 }
 
 // GetSubDomain 获得子域名，为空返回@
-// 阿里云/腾讯云/dnspod/namecheap 需要
+// 阿里云/腾讯云/dnspod/GoDaddy/namecheap 需要
 func (d Domain) GetSubDomain() string {
 	if d.SubDomain != "" {
 		return d.SubDomain

diff --git a/dns/godaddy.go b/dns/godaddy.go
@@ -72,7 +72,7 @@ func (g *GoDaddyDNS) updateDomainRecord(recordType string, ipAddr string, domain
 	for _, domain := range domains {
 		err := g.sendReq(http.MethodPut, recordType, domain, &godaddyRecords{godaddyRecord{
 			Data: ipAddr,
-			Name: domain.SubDomain,
+			Name: domain.GetSubDomain(),
 			TTL:  g.ttl,
 			Type: recordType,
 		}})
@@ -107,7 +107,7 @@ func (g *GoDaddyDNS) sendReq(method string, rType string, domain *config.Domain,
 		}
 	}
 	path := fmt.Sprintf("https://api.godaddy.com/v1/domains/%s/records/%s/%s",
-		domain.DomainName, rType, domain.SubDomain)
+		domain.DomainName, rType, domain.GetSubDomain())
 
 	req, err := http.NewRequest(method, path, body)
 	if err != nil {