fix(cfg): don't load config from file and flags unless required

phe-lab · Nov 16, 2024 · 9b4fbcc · 9b4fbcc
1 parent 0aeb8e2
commit 9b4fbcc
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/cfg/load.go b/cfg/load.go
@@ -33,8 +33,10 @@ func LoadConfig(o interface{}, opt LoadOptions) error {
 	var finalErr error
 
 	// Read from YAML file
-	if err := k.Load(file.Provider(opt.Filename), yaml.Parser()); err != nil {
-		finalErr = err
+	if opt.Filename != "" {
+		if err := k.Load(file.Provider(opt.Filename), yaml.Parser()); err != nil {
+			finalErr = err
+		}
 	}
 
 	// Read from environment variables
@@ -43,8 +45,10 @@ func LoadConfig(o interface{}, opt LoadOptions) error {
 	}
 
 	// Read from command-line flags
-	if err := k.Load(posflag.Provider(opt.Flags, ".", k), nil); err != nil {
-		finalErr = err
+	if opt.Flags != nil {
+		if err := k.Load(posflag.Provider(opt.Flags, ".", k), nil); err != nil {
+			finalErr = err
+		}
 	}
 
 	// Unmarshal config