From f8e015e8e414387b7b319f40e74f2611854d8972 Mon Sep 17 00:00:00 2001 From: zeripath Date: Sun, 10 May 2020 14:30:46 +0100 Subject: [PATCH] Move LFS directory creation out of NewContext (#11362) Fix #9481 (probably others too) Signed-off-by: Andrew Thornton Co-authored-by: Lauris BH --- modules/setting/setting.go | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/modules/setting/setting.go b/modules/setting/setting.go index de0f5cbad..dd7dbd3fd 100644 --- a/modules/setting/setting.go +++ b/modules/setting/setting.go @@ -749,10 +749,6 @@ func NewContext() { LFS.HTTPAuthExpiry = sec.Key("LFS_HTTP_AUTH_EXPIRY").MustDuration(20 * time.Minute) if LFS.StartServer { - if err := os.MkdirAll(LFS.ContentPath, 0700); err != nil { - log.Fatal("Failed to create '%s': %v", LFS.ContentPath, err) - } - LFS.JWTSecretBytes = make([]byte, 32) n, err := base64.RawURLEncoding.Decode(LFS.JWTSecretBytes, []byte(LFS.JWTSecretBase64)) @@ -1111,11 +1107,20 @@ func loadOrGenerateInternalToken(sec *ini.Section) string { return token } +func ensureLFSDirectory() { + if LFS.StartServer { + if err := os.MkdirAll(LFS.ContentPath, 0700); err != nil { + log.Fatal("Failed to create '%s': %v", LFS.ContentPath, err) + } + } +} + // NewServices initializes the services func NewServices() { InitDBConfig() newService() NewLogServices(false) + ensureLFSDirectory() newCacheService() newSessionService() newCORSService()