Feat: Refine error messages to include specific invalid values.

Updated error messages in `server/config/types.go` to provide more context by including the invalid value in the error description. Modified error definitions in `server/errors/errors.go` accordingly. Changed `Learning` field type in `BruteforceSection` to a pointer to `Feature` for consistency.

Signed-off-by: Christian Roessner <[email protected]>

server/config/bruteforce.go

@@ -20,7 +20,7 @@ import "fmt"
 type BruteForceSection struct {
 	IPWhitelist []string         `mapstructure:"ip_whitelist"`
 	Buckets     []BruteForceRule `mapstructure:"buckets"`
-	Learning    []Feature        `mapstructure:"learning"`
+	Learning    []*Feature       `mapstructure:"learning"`
 }
 
 func (b *BruteForceSection) String() string {

server/config/types.go

@@ -16,6 +16,7 @@
 package config
 
 import (
+	"fmt"
 	"strings"
 
 	"github.com/croessner/nauthilus/server/errors"
@@ -64,7 +65,7 @@ func (v *Verbosity) Set(value string) error {
 	case "debug":
 		v.verboseLevel = global.LogLevelDebug
 	default:
-		return errors.ErrWrongVerboseLevel
+		return fmt.Errorf(errors.ErrWrongVerboseLevel.Error(), value)
 	}
 
 	v.name = value
@@ -129,7 +130,7 @@ func (l *LDAPScope) Set(value string) error {
 	case "sub":
 		l.scope = ldap.ScopeWholeSubtree
 	default:
-		return errors.ErrWrongLDAPScope
+		return fmt.Errorf(errors.ErrWrongLDAPScope.Error(), value)
 	}
 
 	l.name = value
@@ -241,7 +242,7 @@ func (b *Backend) Set(value string) error {
 	case global.BackendLuaName:
 		b.backend = global.BackendLua
 	default:
-		return errors.ErrWrongPassDB
+		return fmt.Errorf(errors.ErrWrongPassDB.Error(), value)
 	}
 
 	return nil
@@ -293,7 +294,7 @@ func (f *Feature) Set(value string) error {
 	case global.FeatureTLSEncryption, global.FeatureRBL, global.FeatureRelayDomains, global.FeatureLua, global.FeatureBackendServersMonitoring, global.FeatureBruteForce:
 		f.name = value
 	default:
-		return errors.ErrWrongFeature
+		return fmt.Errorf(errors.ErrWrongFeature.Error(), value)
 	}
 
 	return nil
@@ -378,7 +379,7 @@ func (d *DbgModule) Set(value string) error {
 	case global.DbgFilterName:
 		d.module = global.DbgFilter
 	default:
-		return errors.ErrWrongDebugModule
+		return fmt.Errorf(errors.ErrWrongDebugModule.Error(), value)
 	}
 
 	d.name = value

server/errors/errors.go

@@ -93,11 +93,11 @@ var (
 // env.
 
 var (
-	ErrWrongVerboseLevel = errors.New("wrong verbose level")
-	ErrWrongLDAPScope    = errors.New("wrong LDAP scope")
-	ErrWrongPassDB       = errors.New("wrong passdb backend")
-	ErrWrongFeature      = errors.New("wrong feature")
-	ErrWrongDebugModule  = errors.New("wrong debug module")
+	ErrWrongVerboseLevel = errors.New("wrong verbose level: <%s>")
+	ErrWrongLDAPScope    = errors.New("wrong LDAP scope: <%s>")
+	ErrWrongPassDB       = errors.New("wrong passdb backend: <%s>")
+	ErrWrongFeature      = errors.New("wrong feature: <%s>")
+	ErrWrongDebugModule  = errors.New("wrong debug module: <%s>")
 )
 
 // file.