fix(update): prevent race condition when IP hasn't changed

Author: vorosdev

Makefile

@@ -1,7 +1,7 @@
 # ==============================
 # Variables
 # ==============================
-VERSION       := 0.3.1
+VERSION       := 0.3.2
 BINARY_NAME   := autoip
 COMMIT        := $(shell git rev-parse --short HEAD)
 BUILD_DATE    := $(shell date -u +%Y-%m-%dT%H:%M:%SZ)

commands/create.go

@@ -189,7 +189,7 @@ func RunCreateInteractive() error {
 
 	// ── Resumen final ──
 	printSection("📝 Resumen de configuración")
-	fmt.Println("Proveedor DDNS   :", cfg.DDNS.Provider)
+	fmt.Println("Proveedor DDNS :", cfg.DDNS.Provider)
 	switch cfg.DDNS.Provider {
 	case "noip":
 		fmt.Println("Hostname       :", cfg.DDNS.NoIP.Hostname)

commands/daemon.go

@@ -52,12 +52,13 @@ func runDaemon(ctx context.Context) error {
 		select {
 		case <-ticker.C:
 			log.Info("Revisando cambios de IP...")
-			if err := RunUpdate(ctx); err != nil {
-				log.Error("Error al actualizar IP", "error ", err)
+			err := RunUpdate(ctx)
+			if err != nil {
+				return err
 			}
 
 		case sig := <-sigChan:
-			log.Info("Señal recibida, cerrando daemon ", "signal ", sig)
+			log.Warn("Señal recibida, cerrando daemon ", "signal ", sig)
 			return nil
 
 		case <-ctx.Done():

commands/update.go

@@ -5,13 +5,11 @@ package commands
 
 import (
 	"context"
-	"errors"
 	"fmt"
 
 	"github.com/urfave/cli/v3"
 	"github.com/vorosdev/autoip/config"
 	"github.com/vorosdev/autoip/ddns"
-	"github.com/vorosdev/autoip/ddns/noip"
 	"github.com/vorosdev/autoip/ip"
 	"github.com/vorosdev/autoip/log"
 )
@@ -37,9 +35,10 @@ func RunUpdate(ctx context.Context) error {
 		return nil
 	}
 
-	log.Info(fmt.Sprintf("IP nueva detectada: %s (antes: %s)", ipNueva, ipActual))
+	if ipActual != "" {
+		log.Info(fmt.Sprintf("IP nueva detectada: %s (antes: %s)", ipNueva, ipActual))
+	}
 
-	// Cargar proveedor DDNS dinámicamente
 	provider, err := ddns.LoadProvider(cfg.DDNS)
 	if err != nil {
 		log.Error("Error al cargar proveedor DDNS: ", err)
@@ -48,28 +47,11 @@ func RunUpdate(ctx context.Context) error {
 
 	log.Info("Usando proveedor: ", provider.Name())
 
-	// if err := provider.Update(ipNueva); err != nil {
-	// 	log.Error(fmt.Sprintf("Error al actualizar proveedor %s: ", provider.Name()), err)
-	// 	return err
-	// }
-
-	err = provider.Update(ipNueva)
-	if err != nil {
-		// Validación activa ante respuesta sospechosa
-		if errors.Is(err, noip.ErrNochgSuspicious) {
-			log.Info("Respuesta 'nochg' sospechosa, realizando validación activa de credenciales")
-
-			validateErr := noip.ValidateAuth(cfg.DDNS, "127.0.0.2")
-			if validateErr != nil {
-				log.Error("Validación activa fallida: ", validateErr)
-				return validateErr
-			}
-		} else {
-			log.Error(fmt.Sprintf("Error al actualizar proveedor %s: ", provider.Name()), err)
-			return err
-		}
+	if err := provider.Update(ipNueva); err != nil {
+		log.Error(fmt.Sprintf("Error al actualizar proveedor %s: ", provider.Name()), err)
+		return err
 	}
-	// Guardar la nueva IP
+
 	_ = ip.WriteOldIP(ipActual)
 	_ = ip.WriteCurrentIP(ipNueva)