From 2cf2ca4756be1b7d79a3eba53a09df2a5c41295e Mon Sep 17 00:00:00 2001 From: Abhishek Shroff Date: Fri, 4 Jul 2025 19:51:04 +0530 Subject: [PATCH] [server][storage] Chmod 0o700 instead of error for permissions mismatch --- server/internal/storage/storage.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/server/internal/storage/storage.go b/server/internal/storage/storage.go index e3abb0f3..ff01f6d6 100644 --- a/server/internal/storage/storage.go +++ b/server/internal/storage/storage.go @@ -3,7 +3,6 @@ package storage import ( "encoding/json" "errors" - "fmt" "os" "path/filepath" "strings" @@ -28,13 +27,14 @@ type BackendConfig struct { } func Initialize(db db.Handler) error { - if err := os.MkdirAll(Cfg.Root, 0700); err != nil { + if err := os.MkdirAll(Cfg.Root, 0o700); err != nil { return errors.New("failed to create storage root(" + Cfg.Root + "): " + err.Error()) } else { if stat, err := os.Stat(Cfg.Root); err != nil { return errors.New("failed to stat storage root(" + Cfg.Root + "): " + err.Error()) - } else if stat.Mode()&0xfff != 0700 { - return errors.New("storage root does not have correct permissions (0700): " + fmt.Sprintf("%o", stat.Mode()&0xfff)) + } else if stat.Mode()&0xfff != 0o700 { + os.Chmod(Cfg.Root, 0o700) + // return errors.New("storage root does not have correct permissions: " + fmt.Sprintf("%o", stat.Mode()&0xfff)) } }