From d9667c6252d206769f0fdd1e8f1791dfa9a757a1 Mon Sep 17 00:00:00 2001 From: Alex Date: Sun, 30 May 2021 18:59:39 +0200 Subject: [PATCH] added error checking when parsing ExpiredTime --- modules/serversettings/server_settings.go | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/modules/serversettings/server_settings.go b/modules/serversettings/server_settings.go index bb3bf37..4cf6f56 100644 --- a/modules/serversettings/server_settings.go +++ b/modules/serversettings/server_settings.go @@ -14,10 +14,12 @@ type ServerSettings struct { var Settings ServerSettings func LoadServerSettings() { + log.Debug("load server settings from db") + rows, err := database.DB.Raw("SELECT name, value FROM server_settings").Rows() if err != nil { - log.Fatal("Failed to load server settings:", err) + log.Fatal("failed to load server settings:", err) } defer rows.Close() @@ -29,11 +31,17 @@ func LoadServerSettings() { for rows.Next() { rows.Scan(&name, &value) - log.Infoln(name, value) + + log.Debugln("server setting:", name, value) switch name { case "expired_time": - n, _ := strconv.ParseInt(value, 0, 64) + n, err := strconv.ParseInt(value, 0, 64) + + if err != nil { + log.Fatalln("failed to parse server setting: expired_time", err) + } + Settings.ExpiredTime = n } }