Fixed removing from cache with --with-* flags

cmd/cache.go

@@ -62,11 +62,28 @@ var cacheRemoveCmd = &cobra.Command{
 			assets = append(assets, asset)
 		}
 
-		// add all assets with specified hosts (same host different different ports)
+		// Add all assets with specified hosts (same host different different ports)
+		// This should produce the following SQL:
+		// DELETE FROM magellan_scanned_assets WHERE host=:host
 		for _, host := range withHosts {
-
+			assets = append(assets, magellan.RemoteAsset{
+				Host: host,
+				Port: -1,
+			})
+		}
+		// Add all assets with specified ports (same port different hosts)
+		// This should produce the following SQL:
+		// DELETE FROM magellan_scanned_assets WHERE port=:port
+		for _, port := range withPorts {
+			assets = append(assets, magellan.RemoteAsset{
+				Host: "",
+				Port: port,
+			})
+		}
+		if len(assets) <= 0 {
+			log.Error().Msg("nothing to do")
+			os.Exit(1)
 		}
-		// add all assets with specified ports (same port different hosts)
 		sqlite.DeleteScannedAssets(cachePath, assets...)
 	},
 }

internal/cache/sqlite/sqlite.go

@@ -2,6 +2,7 @@ package sqlite
 
 import (
 	"fmt"
+	"strings"
 
 	"github.com/OpenCHAMI/magellan/internal/util"
 	magellan "github.com/OpenCHAMI/magellan/pkg"
@@ -63,22 +64,36 @@ func InsertScannedAssets(path string, assets ...magellan.RemoteAsset) error {
 }
 
 func DeleteScannedAssets(path string, assets ...magellan.RemoteAsset) error {
+	var (
+		db  *sqlx.DB
+		tx  *sqlx.Tx
+		err error
+	)
 	if assets == nil {
 		return fmt.Errorf("no assets found")
 	}
-	db, err := sqlx.Open("sqlite3", path)
+	db, err = sqlx.Open("sqlite3", path)
 	if err != nil {
 		return fmt.Errorf("failed to open database: %v", err)
 	}
-	tx := db.MustBegin()
+	tx = db.MustBegin()
 	for _, asset := range assets {
+		// skip if neither host nor port are specified
 		if asset.Host == "" && asset.Port <= 0 {
 			continue
 		}
-		sql := fmt.Sprintf(`DELETE FROM %s WHERE port=:port;`, TABLE_NAME)
+		sql := fmt.Sprintf(`DELETE FROM %s`, TABLE_NAME)
-		if asset.Host != "" {
+		where := []string{}
-			sql += "AND host=:host"
+		if asset.Port > 0 {
+			where = append(where, "port=:port")
 		}
+		if asset.Host != "" {
+			where = append(where, "host=:host")
+		}
+		if len(where) <= 0 {
+			continue
+		}
+		sql += fmt.Sprintf(" WHERE %s;", strings.Join(where, " AND "))
 		_, err := tx.NamedExec(sql, &asset)
 		if err != nil {
 			fmt.Printf("failed to execute DELETE transaction: %v\n", err)