Fix: If tagFilter.allowedIds is empty, included should be true.

projectdiscovery · Jul 22, 2023 · 79caf25 · 79caf25
1 parent ddbd8a7
commit 79caf25
Showing 1 changed file with 10 additions and 11 deletions.
diff --git a/v2/pkg/catalog/loader/filter/tag_filter.go b/v2/pkg/catalog/loader/filter/tag_filter.go
@@ -171,18 +171,17 @@ func isIdMatch(tagFilter *TagFilter, templateId string) bool {
 	if len(tagFilter.excludeIds) == 0 && len(tagFilter.allowedIds) == 0 {
 		return true
 	}
-	included := false
-	if len(tagFilter.allowedIds) > 0 {
-		for id := range tagFilter.allowedIds {
-			match, err := filepath.Match(id, templateId)
-			if err != nil {
-				continue
-			}
 
-			if match {
-				included = true
-				break
-			}
+	included := len(tagFilter.allowedIds) == 0
+	for id := range tagFilter.allowedIds {
+		match, err := filepath.Match(id, templateId)
+		if err != nil {
+			break
+		}
+
+		if match {
+			included = true
+			break
 		}
 	}