rename function

Author: DTLP

config.go

@@ -209,7 +209,7 @@ func validateConfig(yamlData []byte) error {
 
 	// check config sections for unexpected keys
 	allowedRepoPoolConfig := getAllowedKeys(mirror.RepoPoolConfig{})
-	if key := validateConfigSection(raw, allowedRepoPoolConfig); key != "" {
+	if key := findUnexpectedKey(raw, allowedRepoPoolConfig); key != "" {
 		return fmt.Errorf("unexpected key: .%v", key)
 	}
 
@@ -220,14 +220,14 @@ func validateConfig(yamlData []byte) error {
 	}
 	allowedDefaults := getAllowedKeys(mirror.DefaultConfig{})
 
-	if key := validateConfigSection(defaultsMap, allowedDefaults); key != "" {
+	if key := findUnexpectedKey(defaultsMap, allowedDefaults); key != "" {
 		return fmt.Errorf("unexpected key: .defaults.%v", key)
 	}
 
 	// check "auth" section in "defaults"
 	if authMap, ok := defaultsMap["auth"].(map[string]interface{}); ok {
 		allowedAuthKeys := getAllowedKeys(mirror.Auth{})
-		if key := validateConfigSection(authMap, allowedAuthKeys); key != "" {
+		if key := findUnexpectedKey(authMap, allowedAuthKeys); key != "" {
 			return fmt.Errorf("unexpected key: .defaults.auth.%v", key)
 		}
 	}
@@ -240,7 +240,7 @@ func validateConfig(yamlData []byte) error {
 			return fmt.Errorf("repositories config section is not valid")
 		}
 
-		if key := validateConfigSection(repoMap, allowedRepoKeys); key != "" {
+		if key := findUnexpectedKey(repoMap, allowedRepoKeys); key != "" {
 			return fmt.Errorf("unexpected key: .repositories[%v].%v", repoMap["remote"], key)
 		}
 
@@ -252,7 +252,7 @@ func validateConfig(yamlData []byte) error {
 			}
 
 			allowedWorktreeKeys := getAllowedKeys(mirror.WorktreeConfig{})
-			if key := validateConfigSection(worktreeMap, allowedWorktreeKeys); key != "" {
+			if key := findUnexpectedKey(worktreeMap, allowedWorktreeKeys); key != "" {
 				return fmt.Errorf("unexpected key: .repositories[%v].worktrees[%v].%v", repoMap["remote"], worktreeMap["link"], key)
 			}
 		}
@@ -261,16 +261,6 @@ func validateConfig(yamlData []byte) error {
 	return nil
 }
 
-func validateConfigSection(raw interface{}, allowedKeys []string) string {
-	for key := range raw.(map[string]interface{}) {
-		if !slices.Contains(allowedKeys, key) {
-			return key
-		}
-	}
-
-	return ""
-}
-
 // getAllowedKeys retrieves a list of allowed keys from the specified struct
 func getAllowedKeys(config interface{}) []string {
 	var allowedKeys []string
@@ -287,6 +277,16 @@ func getAllowedKeys(config interface{}) []string {
 	return allowedKeys
 }
 
+func findUnexpectedKey(raw interface{}, allowedKeys []string) string {
+	for key := range raw.(map[string]interface{}) {
+		if !slices.Contains(allowedKeys, key) {
+			return key
+		}
+	}
+
+	return ""
+}
+
 // diffRepositories will do the diff between current state and new config and
 // return new repositories config and list of remote url which are not found in config
 func diffRepositories(repoPool *mirror.RepoPool, newConfig *mirror.RepoPoolConfig) (