dkirillov/distribution
1
0
Fork 0
forked from TrueCloudLab/distribution
Code Pull requests Activity

Merge pull request #3680 from Jamstah/env-var-clash

Browse source 
Raise a specific error for env var parsing issues
This commit is contained in:
Milos Gajdos 2022-11-07 11:26:09 +00:00 committed by GitHub
commit 08764d51bf
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
configuration/parser.go
View file

@ -138,7 +138,7 @@ func (p *Parser) Parse(in []byte, v interface{}) error {
err = p.overwriteFields(parseAs, pathStr, path[1:], envVar.value) err = p.overwriteFields(parseAs, pathStr, path[1:], envVar.value)
if err != nil { if err != nil {
return err return fmt.Errorf("parsing environment variable %s: %v", pathStr, err)
} }
} }
} }
@ -206,7 +206,6 @@ func (p *Parser) overwriteStruct(v reflect.Value, fullpath string, path []string
fieldVal := reflect.New(sf.Type) fieldVal := reflect.New(sf.Type)
err := yaml.Unmarshal([]byte(payload), fieldVal.Interface()) err := yaml.Unmarshal([]byte(payload), fieldVal.Interface())
if err != nil { if err != nil {
logrus.Warnf("Error parsing environment variable %s: %s", fullpath, err)
return err return err
} }
field.Set(reflect.Indirect(fieldVal)) field.Set(reflect.Indirect(fieldVal))