comment, warning

crowdsecurity · Feb 21, 2025 · 01d21e4 · 01d21e4
1 parent 39bcda3
commit 01d21e4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/pkg/cwhub/item.go b/pkg/cwhub/item.go
@@ -136,7 +136,8 @@ func (i *Item) PathForDownload() (string, error) {
 	path, err := SafePath(i.hub.local.HubDir, i.RemotePath)
 
 	if i.State.IsDownloaded() && path != i.State.DownloadPath {
-		// XXX: item is tainted by also at a different location than expected.
+		// A hub item with the same name is at a different location.
+		// This should not happen.
 		// user is downloading with --force so we are allowed to overwrite but
 		// should we remove the old location from here? Error, warning, more tests?
 		return "", fmt.Errorf("%s is already downloaded at %s", i.FQName(), i.State.DownloadPath)

diff --git a/pkg/cwhub/sync.go b/pkg/cwhub/sync.go
@@ -399,7 +399,7 @@ func (h *Hub) addItemFromSpec(spec *itemSpec) error {
 	}
 
 	if item == nil {
-		h.logger.Infof("Ignoring file %s of type %s", spec.path, spec.ftype)
+		h.logger.Warningf("Ignoring file %s of type %s", spec.path, spec.ftype)
 		return nil
 	}