Fixed config bug for redis and completed config loader with proper error handling.

Author: PythonHacker24

.gitignore

@@ -1,6 +1,9 @@
 # Binary files
 /bin/
 
+# Debug logs
+/logs/
+
 # for macOS dev environments
 .DS_Store
 

cmd/laclm/main.go

@@ -9,9 +9,9 @@ import (
 	"syscall"
 	"time"
 
-	"go.uber.org/zap"
-	"github.com/spf13/cobra"
 	"github.com/MakeNowJust/heredoc"
+	"github.com/spf13/cobra"
+	"go.uber.org/zap"
 
 	"github.com/PythonHacker24/linux-acl-management-backend/api/routes"
 	"github.com/PythonHacker24/linux-acl-management-backend/config"
@@ -40,9 +40,9 @@ func exec() error {
 			`),
 			Run: func(cmd *cobra.Command, args []string) {
 				if configPath != "" {
-					fmt.Printf("Using config file: %s\n", configPath)
+					fmt.Printf("Using config file: %s\n\n", configPath)
 				} else {
-					fmt.Println("No config file provided.")
+					fmt.Println("No config file provided.\n\n")
 				}
 			},
 		}
@@ -61,13 +61,22 @@ func exec() error {
 		load config file
 		if there is an error in loading the config file, then it will exit with code 1
 	*/
-	config.LoadConfig(configPath)
+	if err := config.LoadConfig(configPath); err != nil {
+		fmt.Printf("Configuration Error in %s: %s", 
+			configPath, 
+			err.Error(),
+		)
+		/* since the configuration is invalid, don't proceed */
+		os.Exit(1)
+	}
 
 	/*
 		load environment variables
 		if there is an error or environment variables are not set, then it will exit with code 1
 	*/
 	config.LoadEnv()
+
+	fmt.Println("loaded config")
 
 	/* 
 		true for production, false for development mode 

config.yaml

@@ -4,7 +4,7 @@
 app:
   name: laclm-dev
   version: v1.1
-  debug_mode: development
+  debug_mode: true 
 
 # backend server deployment configs
 server:
@@ -14,13 +14,13 @@ server:
 # databases for operations
 database:
   transaction_logs_redis: 
-    address:
-    password:
-    db: 
+    address: localhost
+    password: testingpassword
+    db: 1
 
 # logging configurations
 logging:
-  file: "logs/app.log"
+  file: logs/app.log
   max_size: 100
   max_backups: 5
   max_age: 30
@@ -29,9 +29,9 @@ logging:
 # filesystem server that needs management
 filesystem_servers:
   - path: /mnt/beegfs-1
-    method: "remote"
+    method: remote
     remote:
-      host: "127.0.0.1"
+      host: 127.0.0.1
       port: 4444
 
 # authentication information

config/database.go

@@ -2,13 +2,14 @@ package config
 
 import (
 	"errors"
+	"fmt"
 
 	"github.com/MakeNowJust/heredoc"
 )
 
 /* database parameters */
 type Database struct {
-	TransactionLogRedis TransactionLogRedis `yaml:"transaction_log_redis"`
+	TransactionLogRedis TransactionLogRedis `yaml:"transaction_logs_redis"`
 }
 
 /* transaction log redis parameters */
@@ -31,6 +32,12 @@ func (r *TransactionLogRedis) Normalize() error {
 			Please check the docs for more information: 
 		`))
 	}
+	
+	/* password can be empty */
+	if r.Password == "" {
+		/* just warn users to use password protected redis */
+		fmt.Println("Prefer using password for redis for security purposes")	
+	}
 
 	if r.DB == "" {
 		r.DB = "0"

config/loader.go

@@ -1,38 +1,48 @@
 package config
 
 import (
+	"fmt"
 	"os"
 
 	"go.uber.org/zap"
 	"gopkg.in/yaml.v3"
+	"github.com/davecgh/go-spew/spew"
 )
 
 /*
 	we need config normalization as well
-	config normalization fixes all the fields that are not present in config file 
+	config normalization fixes all the fields that are not present in config file
 	and sets it to default value
 */
 
 /* loads yaml config file from given file path */
-func LoadConfig(path string) {
+func LoadConfig(path string) error {
 
 	/* read the yaml config file */
     data, err := os.ReadFile(path)
     if err != nil {
-        zap.L().Fatal("Failed to read config file",
-			zap.Error(err),
+		return fmt.Errorf("config loading error %w", 
+			err,
 		)
+
     }
 
 	/* unmarshal the yaml file to defined struct */
     err = yaml.Unmarshal(data, &BackendConfig)
     if err != nil {
-        zap.L().Fatal("Failed to parse YAML config", 
-			zap.Error(err),
+		return fmt.Errorf("config loading error %w", 
+			err,
 		)
     }
+
+	if BackendConfig.AppInfo.DebugMode {
+		fmt.Println("Contents of Config File (debug mode ON)")
+		spew.Dump(BackendConfig)
+		fmt.Println()
+	}
 
-	BackendConfig.Normalize()
+	/* normalize the complete backend config before proceeding */
+	return BackendConfig.Normalize()
 }
 
 /* loads environment variables */

go.mod

@@ -6,6 +6,7 @@ require (
 	github.com/Azure/go-ntlmssp v0.0.0-20221128193559-754e69321358 // indirect
 	github.com/MakeNowJust/heredoc v1.0.0 // indirect
 	github.com/bufbuild/buf v1.53.0 // indirect
+	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/go-asn1-ber/asn1-ber v1.5.8-0.20250403174932-29230038a667 // indirect
 	github.com/go-ldap/ldap/v3 v3.4.11 // indirect
 	github.com/golang-jwt/jwt/v5 v5.2.2 // indirect

go.sum

@@ -5,6 +5,8 @@ github.com/MakeNowJust/heredoc v1.0.0/go.mod h1:mG5amYoWBHf8vpLOuehzbGGw0EHxpZZ6
 github.com/bufbuild/buf v1.53.0 h1:i0OgpDkzv8yLyCokXRCgbQao/SqTnqXYtERre0Jw6Do=
 github.com/bufbuild/buf v1.53.0/go.mod h1:DylfLDMblZt5mX/SEFv2VzZMgdlePArhMGYVpk4M6N0=
 github.com/cpuguy83/go-md2man/v2 v2.0.6/go.mod h1:oOW0eioCTA6cOiMLiUPZOpcVxMig6NIQQ7OS05n1F4g=
+github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
+github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
 github.com/go-asn1-ber/asn1-ber v1.5.8-0.20250403174932-29230038a667 h1:BP4M0CvQ4S3TGls2FvczZtj5Re/2ZzkV9VwqPHH/3Bo=
 github.com/go-asn1-ber/asn1-ber v1.5.8-0.20250403174932-29230038a667/go.mod h1:hEBeB/ic+5LoWskz+yKT7vGhhPYkProFKoKdwZRWMe0=
 github.com/go-ldap/ldap/v3 v3.4.11 h1:4k0Yxweg+a3OyBLjdYn5OKglv18JNvfDykSoI8bW0gU=