chore(namecheap): make code only IPv4 compatible

- Namecheap does not support IPv6 dynamic updates still
qdm12 · Jan 29, 2024 · b771b5c · b771b5c
1 parent 67b83d4
commit b771b5c
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 12 deletions.
diff --git a/internal/provider/errors/validation.go b/internal/provider/errors/validation.go
@@ -18,7 +18,6 @@ var (
 	ErrHostWildcard           = errors.New(`host cannot be a "*"`)
 	ErrIPv4KeyNotSet          = errors.New("IPv4 key is not set")
 	ErrIPv6KeyNotSet          = errors.New("IPv6 key is not set")
-	ErrIPv6NotSupported       = errors.New("IPv6 is not supported by this provider")
 	ErrKeyNotSet              = errors.New("key is not set")
 	ErrKeyNotValid            = errors.New("key is not valid")
 	ErrNameNotSet             = errors.New("name is not set")

diff --git a/internal/provider/provider.go b/internal/provider/provider.go
@@ -130,7 +130,7 @@ func New(providerName models.Provider, data json.RawMessage, domain, host string
 	case constants.LuaDNS:
 		return luadns.New(data, domain, host, ipVersion)
 	case constants.Namecheap:
-		return namecheap.New(data, domain, host, ipVersion)
+		return namecheap.New(data, domain, host)
 	case constants.NameCom:
 		return namecom.New(data, domain, host, ipVersion)
 	case constants.Netcup:

diff --git a/internal/provider/providers/namecheap/provider.go b/internal/provider/providers/namecheap/provider.go
@@ -22,16 +22,12 @@ import (
 type Provider struct {
 	domain        string
 	host          string
-	ipVersion     ipversion.IPVersion
 	password      string
 	useProviderIP bool
 }
 
-func New(data json.RawMessage, domain, host string,
-	ipVersion ipversion.IPVersion) (p *Provider, err error) {
-	if ipVersion == ipversion.IP6 {
-		return nil, fmt.Errorf("%w", errors.ErrIPv6NotSupported)
-	}
+func New(data json.RawMessage, domain, host string) (
+	p *Provider, err error) {
 	extraSettings := struct {
 		Password      string `json:"password"`
 		UseProviderIP bool   `json:"provider_ip"`
@@ -43,7 +39,6 @@ func New(data json.RawMessage, domain, host string,
 	p = &Provider{
 		domain:        domain,
 		host:          host,
-		ipVersion:     ipVersion,
 		password:      extraSettings.Password,
 		useProviderIP: extraSettings.UseProviderIP,
 	}
@@ -65,7 +60,7 @@ func (p *Provider) isValid() error {
 }
 
 func (p *Provider) String() string {
-	return utils.ToString(p.domain, p.host, constants.Namecheap, p.ipVersion)
+	return utils.ToString(p.domain, p.host, constants.Namecheap, ipversion.IP4)
 }
 
 func (p *Provider) Domain() string {
@@ -77,7 +72,11 @@ func (p *Provider) Host() string {
 }
 
 func (p *Provider) IPVersion() ipversion.IPVersion {
-	return p.ipVersion
+	return ipversion.IP4
+}
+
+func (p *Provider) IPv6Suffix() netip.Prefix {
+	return netip.Prefix{}
 }
 
 func (p *Provider) Proxied() bool {
@@ -93,7 +92,7 @@ func (p *Provider) HTML() models.HTMLRow {
 		Domain:    fmt.Sprintf("<a href=\"http://%s\">%s</a>", p.BuildDomainName(), p.BuildDomainName()),
 		Host:      p.Host(),
 		Provider:  "<a href=\"https://namecheap.com\">Namecheap</a>",
-		IPVersion: p.ipVersion.String(),
+		IPVersion: ipversion.IP4.String(),
 	}
 }