up

rest/controllers/cloudflare_controller.go

@@ -2,12 +2,28 @@ package controllers
 
 import (
 	"strings"
+	"unicode"
 
 	"github.com/gofiber/fiber/v2"
 	"github.com/sujit-baniya/fiber-boilerplate/pkg/models"
 	"github.com/sujit-baniya/fiber-boilerplate/pkg/services"
 )
 
+// sanitizeCFToken elimina espacios, saltos de línea, BOM y cualquier caracter
+// no imprimible que pueda romper la autenticación Bearer de Cloudflare.
+func sanitizeCFToken(s string) string {
+	// Eliminar BOM UTF-8 si existe
+	s = strings.TrimPrefix(s, "\xef\xbb\xbf")
+	// Filtrar solo rómanos imprimibles (printable ASCII, excluye control chars)
+	var b strings.Builder
+	for _, r := range s {
+		if r > 32 && r < 127 && unicode.IsPrint(r) {
+			b.WriteRune(r)
+		}
+	}
+	return b.String()
+}
+
 // ─── Configuración ────────────────────────────────────────────────────────────
 
 // CloudflareConfigPage renderiza la vista de gestión de Cloudflare.
@@ -41,8 +57,8 @@ func SaveCloudflareConfig(c *fiber.Ctx) error {
 	}
 
 	cfg := models.CloudflareConfig{
-		APIToken:  strings.TrimSpace(b.APIToken),
-		AccountID: strings.TrimSpace(b.AccountID),
+		APIToken:  sanitizeCFToken(b.APIToken),
+		AccountID: sanitizeCFToken(b.AccountID),
 		Nota:      b.Nota,
 	}
 	cfg.ID = b.ID
@@ -55,12 +71,34 @@ func SaveCloudflareConfig(c *fiber.Ctx) error {
 
 // ─── Helper ───────────────────────────────────────────────────────────────────
 
+// CleanCloudflareToken re-guarda el token activo aplicando saneamiento.
+// Útil cuando el token fue guardado antes del fix de caracteres invisibles.
+func CleanCloudflareToken(c *fiber.Ctx) error {
+	cfg, err := models.GetCloudflareConfig()
+	if err != nil {
+		return c.Status(fiber.StatusServiceUnavailable).JSON(fiber.Map{"error": "Sin configuración activa"})
+	}
+	before := len([]rune(cfg.APIToken))
+	cfg.APIToken = sanitizeCFToken(cfg.APIToken)
+	cfg.AccountID = sanitizeCFToken(cfg.AccountID)
+	after := len([]rune(cfg.APIToken))
+	if err := models.SaveCloudflareConfig(*cfg); err != nil {
+		return c.Status(fiber.StatusInternalServerError).JSON(fiber.Map{"error": err.Error()})
+	}
+	return c.JSON(fiber.Map{
+		"message":      "Token re-guardado limpio",
+		"chars_before": before,
+		"chars_after":  after,
+		"chars_removed": before - after,
+	})
+}
+
 func cloudflareClient() (*services.CloudflareClient, error) {
 	cfg, err := models.GetCloudflareConfig()
 	if err != nil {
 		return nil, err
 	}
-	return services.NewCloudflareClient(strings.TrimSpace(cfg.APIToken), strings.TrimSpace(cfg.AccountID)), nil
+	return services.NewCloudflareClient(sanitizeCFToken(cfg.APIToken), sanitizeCFToken(cfg.AccountID)), nil
 }
 
 // VerifyCloudflareToken verifica que el token activo sea válido contra la API de Cloudflare.
@@ -72,16 +110,29 @@ func VerifyCloudflareToken(c *fiber.Ctx) error {
 			"error": "No hay configuración activa de Cloudflare",
 		})
 	}
-	token := strings.TrimSpace(cfg.APIToken)
+	raw := cfg.APIToken
+	token := sanitizeCFToken(raw)
 	if token == "" {
 		return c.Status(fiber.StatusBadRequest).JSON(fiber.Map{"ok": false, "error": "Token vacío"})
 	}
-	client := services.NewCloudflareClient(token, strings.TrimSpace(cfg.AccountID))
+	// Si la longitud cambia después de sanear, había caracteres basura
+	dirty := len([]rune(raw)) != len([]rune(token))
+	client := services.NewCloudflareClient(token, sanitizeCFToken(cfg.AccountID))
 	result, err := client.VerifyToken()
 	if err != nil {
-		return c.Status(fiber.StatusInternalServerError).JSON(fiber.Map{"ok": false, "error": err.Error()})
+		return c.Status(fiber.StatusInternalServerError).JSON(fiber.Map{
+			"ok":         false,
+			"error":      err.Error(),
+			"token_len":  len(token),
+			"was_dirty":  dirty,
+		})
 	}
-	return c.JSON(fiber.Map{"ok": true, "data": result})
+	return c.JSON(fiber.Map{
+		"ok":        true,
+		"data":      result,
+		"token_len": len(token),
+		"was_dirty": dirty,
+	})
 }
 
 // ─── Endpoints de datos ───────────────────────────────────────────────────────