Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Commit 582f377

Browse files
committed
Fix invalid volume definition
It can have more than one `:` in its definition. Signed-off-by: Djordje Lukic <[email protected]>
1 parent cac8c32 commit 582f377

File tree

3 files changed

+2
-4
lines changed

3 files changed

+2
-4
lines changed

internal/packager/extract.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,8 +76,7 @@ func Extract(name string, ops ...func(*types.App) error) (*types.App, error) {
7676
types.WithPath(appname),
7777
types.WithSource(types.AppSourceSplit),
7878
)
79-
app, err := loader.LoadFromDirectory(appname, appOpts...)
80-
return app, err
79+
return loader.LoadFromDirectory(appname, appOpts...)
8180
}
8281
// not a dir: a tarball package, extract that in a temp dir
8382
app, err := loader.LoadFromTar(appname, ops...)

internal/validator/rules/externalsecrets.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ func NewExternalSecretsRule() Rule {
1212
}
1313

1414
func (s *externalSecretsValidator) Collect(parent string, key string, value interface{}) {
15-
1615
}
1716

1817
func (s *externalSecretsValidator) Accept(parent string, key string) bool {

internal/validator/rules/relativepath.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ func (s *relativePathRule) Validate(value interface{}) []error {
5353
}
5454

5555
parts := strings.Split(str, ":")
56-
if len(parts) != 2 {
56+
if len(parts) <= 1 {
5757
errs = append(errs, fmt.Errorf("invalid volume definition (%q) in service %q", str, s.service))
5858
continue
5959
}

0 commit comments

Comments
 (0)