cleanup debugging

aaronhurt · Jul 12, 2024 · 82acd29 · 82acd29
1 parent 3fe446b
commit 82acd29
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 10 deletions.
diff --git a/command/update/update.go b/command/update/update.go
@@ -2,6 +2,7 @@
 package update
 
 import (
+	// backend definitions
 	"github.com/leprechau/ipman/internal/dns"
 	"github.com/leprechau/ipman/internal/ip"
 )
@@ -33,7 +34,6 @@ func (c *Command) checkUpdate(iType ip.IFlag, rType dns.RType) error {
 	var local, remote, record string
 	var err error
 
-	c.Log.Debug("checking update", "config", c.config)
 	// get local address
 	if local, err = c.ip.Get(iType); err != nil {
 		return err
@@ -47,15 +47,14 @@ func (c *Command) checkUpdate(iType ip.IFlag, rType dns.RType) error {
 	c.Log.Debug("successfully queried local and remote addresses", "iType", iType, "local", local, "remote", remote)
 
 	// update if needed
-	//	if local != remote {
-	// attempt to update remote record
-	if record, err = c.dns.Upsert(c.config.zone, c.config.name, local, rType); err != nil {
-		return err
+	if local != remote {
+		// attempt to update remote record
+		if record, err = c.dns.Upsert(c.config.zone, c.config.name, local, rType); err != nil {
+			return err
+		}
+		c.Log.Info("updated remote", "record", record, "rType", rType, "data", local)
 	}
 
-	c.Log.Info("updated remote", "record", record, "rType", rType, "data", local)
-	//	}
-
 	// all okay
 	return nil
 }
diff --git a/internal/dns/cloudflare/cloudflare.go b/internal/dns/cloudflare/cloudflare.go
@@ -21,7 +21,6 @@ func (c *Config) Get(zone, name string, typ dns.RType) (string, error) {
 			SetPathParam("zone", zone).
 			SetResult(&ZoneResponse{}).
 			SetError(&DNSErrorResponse{}).
-			SetDebug(true).
 			Get("/zones/{zone}"); err != nil {
 			return "", err
 		}
@@ -45,7 +44,6 @@ func (c *Config) Get(zone, name string, typ dns.RType) (string, error) {
 		SetPathParam("zone", zone).
 		SetResult(&DNSResponse{}).
 		SetError(&DNSErrorResponse{}).
-		SetDebug(true).
 		Get("/zones/{zone}/dns_records"); err != nil {
 		return "", err
 	}