Fixed empty path validation

ewoutp · ewoutp · commit 53c7c74e55b3 · 2018-03-29T16:15:58.000+02:00
diff --git a/pkg/apis/storage/v1alpha/local_storage_spec.go b/pkg/apis/storage/v1alpha/local_storage_spec.go
@@ -45,6 +45,11 @@ func (s LocalStorageSpec) Validate() error {
 	if len(s.LocalPath) == 0 {
 		return maskAny(errors.Wrapf(ValidationError, "localPath cannot be empty"))
 	}
+	for _, p := range s.LocalPath {
+		if len(p) == 0 {
+			return maskAny(errors.Wrapf(ValidationError, "localPath cannot contain empty strings"))
+		}
+	}
 	return nil
 }
 

Original file line number	Diff line number	Diff line change
`@@ -45,6 +45,11 @@ func (s LocalStorageSpec) Validate() error {`
`45`	`45`	`if len(s.LocalPath) == 0 {`
`46`	`46`	`return maskAny(errors.Wrapf(ValidationError, "localPath cannot be empty"))`
`47`	`47`	`}`
	`48`	`+ for _, p := range s.LocalPath {`
	`49`	`+ if len(p) == 0 {`
	`50`	`+ return maskAny(errors.Wrapf(ValidationError, "localPath cannot contain empty strings"))`
	`51`	`+ }`
	`52`	`+ }`
`48`	`53`	`return nil`
`49`	`54`	`}`
`50`	`55`