From 208bba71919977e92af4f8ff65bc4792a21202e4 Mon Sep 17 00:00:00 2001 From: Sascha Grunert Date: Wed, 26 Apr 2023 11:53:44 +0200 Subject: [PATCH] Fix bug in subscriptions file save We have to keep the `filepath.Dir` since it has been added on purpose. Signed-off-by: Sascha Grunert --- pkg/subscriptions/subscriptions.go | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/pkg/subscriptions/subscriptions.go b/pkg/subscriptions/subscriptions.go index c21f5ecc1..f956c8274 100644 --- a/pkg/subscriptions/subscriptions.go +++ b/pkg/subscriptions/subscriptions.go @@ -37,10 +37,11 @@ type subscriptionData struct { // saveTo saves subscription data to given directory func (s subscriptionData) saveTo(dir string) error { - if err := umask.MkdirAllIgnoreUmask(dir, s.dirMode); err != nil { + path := filepath.Join(dir, s.name) + if err := umask.MkdirAllIgnoreUmask(filepath.Dir(path), s.dirMode); err != nil { return fmt.Errorf("create subscription directory: %w", err) } - if err := umask.WriteFileIgnoreUmask(filepath.Join(dir, s.name), s.data, s.mode); err != nil { + if err := umask.WriteFileIgnoreUmask(path, s.data, s.mode); err != nil { return fmt.Errorf("write subscription data: %w", err) } return nil