go: modernize

Author: saltydk

internal/config/accounts.go

@@ -161,8 +161,8 @@ func ValidateConfig(config *Config, inputMap map[string]any) error {
 	for key := range inputMap {
 		logging.DebugBool(verboseMode, "ValidateConfig - checking key '%s'", key)
 		found := false
-		for i := 0; i < configType.NumField(); i++ {
-			field := configType.Field(i)
+		for field := range configType.Fields() {
+			field := field
 			yamlTag := field.Tag.Get("yaml")
 			logging.DebugBool(verboseMode, "ValidateConfig - comparing key '%s' with field '%s' (YAML tag: '%s')", key, field.Name, yamlTag)
 			// Handle inline YAML tags
@@ -184,8 +184,8 @@ func ValidateConfig(config *Config, inputMap map[string]any) error {
 		for key := range userMap {
 			logging.DebugBool(verboseMode, "ValidateConfig - checking key '%s' in 'user' section", key)
 			found := false
-			for i := 0; i < userType.NumField(); i++ {
-				field := userType.Field(i)
+			for field := range userType.Fields() {
+				field := field
 				yamlTag := field.Tag.Get("yaml")
 				logging.DebugBool(verboseMode, "ValidateConfig - comparing key '%s' with user field '%s' (YAML tag: '%s')", key, field.Name, yamlTag)
 				if yamlTag == key || (strings.Contains(yamlTag, ",") && strings.Split(yamlTag, ",")[0] == key) {
@@ -220,8 +220,8 @@ func ValidateConfig(config *Config, inputMap map[string]any) error {
 		for key := range cfMap {
 			logging.DebugBool(verboseMode, "ValidateConfig - checking key '%s' in 'cloudflare' section", key)
 			found := false
-			for i := 0; i < cfType.NumField(); i++ {
-				field := cfType.Field(i)
+			for field := range cfType.Fields() {
+				field := field
 				yamlTag := field.Tag.Get("yaml")
 				logging.DebugBool(verboseMode, "ValidateConfig - comparing key '%s' with cloudflare field '%s' (YAML tag: '%s')", key, field.Name, yamlTag)
 				if yamlTag == key || (strings.Contains(yamlTag, ",") && strings.Split(yamlTag, ",")[0] == key) {
@@ -242,8 +242,8 @@ func ValidateConfig(config *Config, inputMap map[string]any) error {
 		for key := range dhMap {
 			logging.DebugBool(verboseMode, "ValidateConfig - checking key '%s' in 'dockerhub' section", key)
 			found := false
-			for i := 0; i < dhType.NumField(); i++ {
-				field := dhType.Field(i)
+			for field := range dhType.Fields() {
+				field := field
 				yamlTag := field.Tag.Get("yaml")
 				logging.DebugBool(verboseMode, "ValidateConfig - comparing key '%s' with dockerhub field '%s' (YAML tag: '%s')", key, field.Name, yamlTag)
 				if yamlTag == key || (strings.Contains(yamlTag, ",") && strings.Split(yamlTag, ",")[0] == key) {

internal/config/generic.go

@@ -110,8 +110,8 @@ func checkExtraFieldsInternal(inputMap map[string]any, config any, context strin
 	for key := range inputMap {
 		logging.DebugBool(verboseMode, "Checking for extra field: %s", key)
 		found := false
-		for i := 0; i < configType.NumField(); i++ {
-			field := configType.Field(i)
+		for field := range configType.Fields() {
+			field := field
 			yamlTag := field.Tag.Get("yaml")
 			yamlKey := strings.Split(yamlTag, ",")[0]
 			if yamlKey == key {