refactor: simplify config reader and improve logging

This commit is contained in:
garionion 2025-02-27 15:22:17 +01:00 committed by garionion (aider)
parent 364a27e44f
commit 540c69d7d6

View file

@ -62,7 +62,7 @@ var db *sql.DB
func main() { func main() {
// Use the global cfg variable instead of declaring a new one // Use the global cfg variable instead of declaring a new one
cfgReader := config.NewConfReader("config").WithSearchDirs("./", ".") cfgReader := config.NewConfReader("config")
cfgErr := cfgReader.Read(&cfg) cfgErr := cfgReader.Read(&cfg)
zapcfg := zap.NewProductionConfig() zapcfg := zap.NewProductionConfig()
@ -79,8 +79,10 @@ func main() {
logger.Fatal("Failed to load config", zap.Error(cfgErr)) logger.Fatal("Failed to load config", zap.Error(cfgErr))
} }
logger.Debug("Loaded config", zap.Any("config", cfg))
// Add detailed logging of the OAuth config // Add detailed logging of the OAuth config
logger.Info("OAuth Configuration", logger.Info("OAuth Configuration",
zap.String("ClientID", cfg.OAuth.ClientID), zap.String("ClientID", cfg.OAuth.ClientID),
zap.String("ClientSecret", cfg.OAuth.ClientSecret), zap.String("ClientSecret", cfg.OAuth.ClientSecret),
zap.String("AuthURL", cfg.OAuth.AuthURL), zap.String("AuthURL", cfg.OAuth.AuthURL),
@ -98,7 +100,7 @@ func main() {
RedirectURL: fmt.Sprintf("http://localhost:%d/oauth/callback", cfg.Server.Port), RedirectURL: fmt.Sprintf("http://localhost:%d/oauth/callback", cfg.Server.Port),
Scopes: []string{"profile", "email"}, Scopes: []string{"profile", "email"},
} }
// Log the OAuth config that's actually being used // Log the OAuth config that's actually being used
logger.Info("Initialized OAuth Config", logger.Info("Initialized OAuth Config",
zap.String("ClientID", oauthConfig.ClientID), zap.String("ClientID", oauthConfig.ClientID),