fix: derive bouncer Valid status from revoked field (CrowdSec >= v1.7.x)

CrowdSec v1.7.x replaced the "valid" JSON field with "revoked: false"
in the bouncers list API response. The manager still reads "valid",
which no longer exists, causing all bouncers to show as "disconnected".

This fix reads the "revoked" field and derives Valid = !Revoked.
Falls back to treating the bouncer as valid if neither field is present.

Fixes #47

Author: strausmann

internal/api/handlers/health_diagnostics.go

@@ -49,6 +49,22 @@ func parseBouncersJSON(bouncerOutput string, computeStatus bool) ([]models.Bounc
 			bouncer.Version = version
 		}
 
+		// CrowdSec >= v1.7.x replaced "valid: true" with "revoked: false".
+		// Read the revoked field; if present, derive Valid from it.
+		if revoked, err := jsonparser.GetBoolean(value, "revoked"); err == nil {
+			bouncer.Revoked = revoked
+			bouncer.Valid = !revoked
+		}
+		// If neither "valid" nor "revoked" was found (edge case),
+		// treat the bouncer as valid — it exists in the list, so it was not deleted.
+		if !bouncer.Valid {
+			if _, validErr := jsonparser.GetBoolean(value, "valid"); validErr != nil {
+				if _, revokedErr := jsonparser.GetBoolean(value, "revoked"); revokedErr != nil {
+					bouncer.Valid = true
+				}
+			}
+		}
+
 		if computeStatus {
 			if !bouncer.Valid {
 				bouncer.Status = "disconnected"

internal/models/models.go

@@ -149,6 +149,7 @@ func (d *DecisionRaw) Normalize() Decision {
 type Bouncer struct {
 	Name      string    `json:"name"`
 	IPAddress string    `json:"ip_address"`
+	Revoked   bool      `json:"revoked"`
 	Valid     bool      `json:"valid"`
 	LastPull  time.Time `json:"last_pull"`
 	Type      string    `json:"type"`