111

Author: lvan100

gs/log.go

@@ -28,19 +28,21 @@ import (
 // initLog initializes the application's logging system.
 func initLog() error {
 
-	// Refresh the global system configuration.
+	// Step 1: Refresh the global system configuration.
 	p, err := new(gs_conf.SysConfig).Refresh()
 	if err != nil {
 		return util.FormatError(err, "refresh error in source sys")
 	}
 
+	// Step 2: Load logging-related configuration parameters.
 	var c struct {
 		// LocalDir is the directory that contains configuration files.
 		// Defaults to "./conf" if not provided.
 		LocalDir string `value:"${spring.app.config-local.dir:=./conf}"`
 
 		// Profiles specifies the active application profile(s),
-		// such as "dev" or "prod".
+		// such as "dev", "prod", etc.
+		// Multiple profiles can be provided as a comma-separated list.
 		Profiles string `value:"${spring.profiles.active:=}"`
 	}
 	if err = p.Bind(&c); err != nil {
@@ -49,9 +51,10 @@ func initLog() error {
 
 	extensions := []string{".properties", ".yaml", ".yml", ".xml", ".json"}
 
+	// Step 3: Build a list of candidate configuration files.
 	var files []string
 	if profiles := strings.TrimSpace(c.Profiles); profiles != "" {
-		for s := range strings.SplitSeq(profiles, ",") {
+		for s := range strings.SplitSeq(profiles, ",") { // NOTE: range returns index
 			if s = strings.TrimSpace(s); s != "" {
 				for _, ext := range extensions {
 					files = append(files, filepath.Join(c.LocalDir, "log-"+s+ext))
@@ -63,25 +66,24 @@ func initLog() error {
 		files = append(files, filepath.Join(c.LocalDir, "log"+ext))
 	}
 
-	// Determine which log configuration file to use.
+	// Step 4: Detect existing configuration files.
 	var logFiles []string
 	for _, s := range files {
 		if ok, err := util.PathExists(s); err != nil {
 			return err
-		} else if !ok {
-			continue
+		} else if ok {
+			logFiles = append(logFiles, s)
 		}
-		logFiles = append(logFiles, s)
 	}
 
-	// If no configuration file exists, leave the logger as default.
-	if n := len(logFiles); n == 0 {
+	// Step 5: Apply logging configuration or fall back to defaults.
+	switch n := len(logFiles); {
+	case n == 0:
 		log.Infof(nil, log.TagAppDef, "no log configuration file found, using default logger")
 		return nil
-	} else if n > 1 {
+	case n > 1:
 		return util.FormatError(nil, "multiple log files found: %s", logFiles)
+	default:
+		return log.RefreshFile(logFiles[0])
 	}
-
-	// Refresh the logger configuration from the selected file.
-	return log.RefreshFile(logFiles[0])
 }