Catch whitespace csv.spec.provider.name values (#272)

komish · web-flow · commit e6998ff1feb3 · 2024-01-10T14:28:16.000Z
Signed-off-by: Jose R. Gonzalez &lt;jose@flutes.dev&gt;
diff --git a/pkg/validation/internal/operatorhub.go b/pkg/validation/internal/operatorhub.go
@@ -296,7 +296,7 @@ func checkSpecMaintainers(checks CSVChecks) CSVChecks {
 
 // checkSpecProviderName will validate the values informed via csv.Spec.Provider.Name
 func checkSpecProviderName(checks CSVChecks) CSVChecks {
-	if checks.csv.Spec.Provider.Name == "" {
+	if strings.TrimSpace(checks.csv.Spec.Provider.Name) == "" {
 		checks.errs = append(checks.errs, fmt.Errorf("csv.Spec.Provider.Name not specified"))
 	}
 	return checks

Original file line number	Diff line number	Diff line change
`@@ -296,7 +296,7 @@ func checkSpecMaintainers(checks CSVChecks) CSVChecks {`
`296`	`296`
`297`	`297`	`// checkSpecProviderName will validate the values informed via csv.Spec.Provider.Name`
`298`	`298`	`func checkSpecProviderName(checks CSVChecks) CSVChecks {`
`299`		`- if checks.csv.Spec.Provider.Name == "" {`
	`299`	`+ if strings.TrimSpace(checks.csv.Spec.Provider.Name) == "" {`
`300`	`300`	`checks.errs = append(checks.errs, fmt.Errorf("csv.Spec.Provider.Name not specified"))`
`301`	`301`	`}`
`302`	`302`	`return checks`