rnsrk/wisski-cloud-distillery
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

config: Handle errors better

Browse source
This commit is contained in:
Tom Wiesing 2022-11-23 15:27:58 +01:00
parent 8997f54a8c
commit 55d5c9c529
No known key found for this signature in database
4 changed files with 28 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

8
internal/config/read.go
View file

@ -7,6 +7,7 @@ import (
"github.com/FAU-CDI/wisski-distillery/pkg/environment"
"github.com/FAU-CDI/wisski-distillery/pkg/envreader"
"github.com/FAU-CDI/wisski-distillery/pkg/stringparser"
"github.com/pkg/errors"
)
// Unmarshal updates this configuration from the provided [io.Reader].
@ -45,15 +46,14 @@ func (config *Config) Unmarshal(env environment.Environment, src io.Reader) erro
// read the value with a default
value, ok := values[tEnv]
if !ok || value == "" {
if tDefault == "" {
continue
if tDefault != "" {
value = tDefault
}
value = tDefault
}
// parse the value!
if err := stringparser.Parse(env, tParser, value, vField); err != nil {
return err
return errors.Errorf("Config.Unmarshal: Setting %q, Parser %q: %s", tEnv, tParser, err)
}
}