From 44b518a5b2f639e7a7eaa0ffde4aa057943996eb Mon Sep 17 00:00:00 2001 From: jamesread Date: Thu, 30 Oct 2025 13:04:41 +0000 Subject: [PATCH] fix: panic when loading sessions.yaml --- service/internal/auth/sessions.go | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/service/internal/auth/sessions.go b/service/internal/auth/sessions.go index a78fcd3..43ab930 100644 --- a/service/internal/auth/sessions.go +++ b/service/internal/auth/sessions.go @@ -85,21 +85,21 @@ func LoadUserSessions(cfg *config.Config) { sessionStorageMutex.Lock() defer sessionStorageMutex.Unlock() - data, err := os.ReadFile(cfg.GetDir() + "/sessions.yaml") - if err != nil { - logrus.WithError(err).Warn("Failed to read sessions.yaml file") - // Always reset in-memory sessions on load error - sessionStorage = &SessionStorage{Providers: make(map[string]*SessionProvider)} - return - } + data, err := os.ReadFile(cfg.GetDir() + "/sessions.yaml") + if err != nil { + logrus.WithError(err).Warn("Failed to read sessions.yaml file") + // Always reset in-memory sessions on load error + sessionStorage = &SessionStorage{Providers: make(map[string]*SessionProvider)} + return + } - err = yaml.Unmarshal(data, &sessionStorage) - if err != nil { - logrus.WithError(err).Error("Failed to unmarshal sessions.yaml") - // Always reset in-memory sessions on parse error - sessionStorage = &SessionStorage{Providers: make(map[string]*SessionProvider)} - return - } + err = yaml.Unmarshal(data, &sessionStorage) + if err != nil { + logrus.WithError(err).Error("Failed to unmarshal sessions.yaml") + // Always reset in-memory sessions on parse error + sessionStorage = &SessionStorage{Providers: make(map[string]*SessionProvider)} + return + } // Ensure sessionStorage and Providers are properly initialized if sessionStorage == nil {