diff options
| author | Raúl Benencia <id@rbenencia.name> | 2026-06-05 15:41:09 -0300 |
|---|---|---|
| committer | Raul Benencia <46945030+raul-te@users.noreply.github.com> | 2026-06-05 16:29:33 -0300 |
| commit | 78cfb30c7903c70f423256f55d7133609f0c2c8e (patch) | |
| tree | 688ccd0e070f0bef5b0b9490ac0d75e823d02dac /internal/environment/flags.go | |
| parent | 3c11829324a7151212d17bacc5d403e6d3a2204a (diff) | |
Return flag validation errors
Diffstat (limited to 'internal/environment/flags.go')
| -rw-r--r-- | internal/environment/flags.go | 19 |
1 files changed, 8 insertions, 11 deletions
diff --git a/internal/environment/flags.go b/internal/environment/flags.go index 09c243e..2ffda31 100644 --- a/internal/environment/flags.go +++ b/internal/environment/flags.go @@ -15,6 +15,7 @@ package environment import ( + "errors" "flag" "fmt" "os" @@ -201,23 +202,19 @@ func (env *Environment) setConfigValue(key, value string) error { return nil } -func (env *Environment) validateFlags(flags *flag.FlagSet) { - error := false +func (env *Environment) validateFlags() error { + var messages []string if env.DataDir == "" { - fmt.Println("[*] You must specify the data-dir parameter") - error = true + messages = append(messages, "[*] You must specify the data-dir parameter") } if env.StaticDir == "" { - fmt.Println("[*] You must specify the data-dir parameter") - error = true + messages = append(messages, "[*] You must specify the data-dir parameter") } - if error { - fmt.Println("\nAvailable parameters:") - flags.PrintDefaults() - fmt.Println("\nParameters can be specified as environment variables, arguments or in a config file.") - os.Exit(1) + if len(messages) > 0 { + return errors.New(strings.Join(messages, "\n")) } + return nil } |
