validate settings for c4gh.keys as a temp solution

sda/internal/config/config.go

@@ -367,7 +367,27 @@ func NewConfig(app string) (*Config, error) {
 		requiredConfVars = []string{
 			"c4gh.filepath",
 			"c4gh.passphrase",
-			"c4gh.keys",
+		}
+		// This should be refactored once the old method for reading a single key file is removed
+		if viper.IsSet("c4gh.keys") {
+			// Check if at least one key entry has a valid filepath and passphrase
+			var keyConfigs []map[string]string
+			if err := viper.UnmarshalKey("c4gh.keys", &keyConfigs); err != nil {
+				return nil, fmt.Errorf("failed to parse key configurations: %v", err)
+			}
+
+			atLeastOneKeySet := false
+			for _, key := range keyConfigs {
+				if key["filepath"] != "" && key["passphrase"] != "" {
+					atLeastOneKeySet = true
+
+					break
+				}
+			}
+
+			if !atLeastOneKeySet {
+				return nil, fmt.Errorf("at least one valid entry in c4gh.keys must have a filepath and passphrase set")
+			}
 		}
 	case "s3inbox":
 		requiredConfVars = []string{
@@ -461,28 +481,7 @@ func NewConfig(app string) (*Config, error) {
 	}
 
 	for _, s := range requiredConfVars {
-		if s == "c4gh.keys" {
-			if viper.IsSet("c4gh.keys") {
-				// Check if at least one key entry has a valid filepath and passphrase
-				var keyConfigs []map[string]string
-				if err := viper.UnmarshalKey("c4gh.keys", &keyConfigs); err != nil {
-					return nil, fmt.Errorf("failed to parse key configurations: %v", err)
-				}
-
-				atLeastOneKeySet := false
-				for _, key := range keyConfigs {
-					if key["filepath"] != "" && key["passphrase"] != "" {
-						atLeastOneKeySet = true
-
-						break
-					}
-				}
-
-				if !atLeastOneKeySet {
-					return nil, fmt.Errorf("at least one valid entry in c4gh.keys must have a filepath and passphrase set")
-				}
-			}
-		} else if !viper.IsSet(s) {
+		if !viper.IsSet(s) {
 			return nil, fmt.Errorf("%s not set", s)
 		}
 	}