fix: boolean flag parsing with map.
This commit is contained in:
parent
b55be9fdea
commit
b4c7b90c9e
2 changed files with 39 additions and 4 deletions
|
@ -3,6 +3,7 @@ package flag
|
||||||
import (
|
import (
|
||||||
"fmt"
|
"fmt"
|
||||||
"reflect"
|
"reflect"
|
||||||
|
"regexp"
|
||||||
"strings"
|
"strings"
|
||||||
|
|
||||||
"github.com/containous/traefik/v2/pkg/config/parser"
|
"github.com/containous/traefik/v2/pkg/config/parser"
|
||||||
|
@ -80,8 +81,8 @@ func (f *flagSet) parseOne() (bool, error) {
|
||||||
return true, nil
|
return true, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
n := strings.ToLower(name)
|
flagType := f.getFlagType(name)
|
||||||
if f.flagTypes[n] == reflect.Bool || f.flagTypes[n] == reflect.Ptr {
|
if flagType == reflect.Bool || flagType == reflect.Ptr {
|
||||||
f.setValue(name, "true")
|
f.setValue(name, "true")
|
||||||
return true, nil
|
return true, nil
|
||||||
}
|
}
|
||||||
|
@ -111,10 +112,30 @@ func (f *flagSet) setValue(name string, value string) {
|
||||||
}
|
}
|
||||||
|
|
||||||
v, ok := f.values[key]
|
v, ok := f.values[key]
|
||||||
if ok && f.flagTypes[strings.ToLower(name)] == reflect.Slice {
|
if ok && f.getFlagType(name) == reflect.Slice {
|
||||||
f.values[key] = v + "," + value
|
f.values[key] = v + "," + value
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
f.values[key] = value
|
f.values[key] = value
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func (f *flagSet) getFlagType(name string) reflect.Kind {
|
||||||
|
neutral := strings.ToLower(name)
|
||||||
|
|
||||||
|
kind, ok := f.flagTypes[neutral]
|
||||||
|
if ok {
|
||||||
|
return kind
|
||||||
|
}
|
||||||
|
|
||||||
|
for n, k := range f.flagTypes {
|
||||||
|
if strings.Contains(n, parser.MapNamePlaceholder) {
|
||||||
|
p := strings.NewReplacer(".", `\.`, parser.MapNamePlaceholder, `([^.]+)`).Replace(n)
|
||||||
|
if regexp.MustCompile(p).MatchString(neutral) {
|
||||||
|
return k
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return reflect.Invalid
|
||||||
|
}
|
||||||
|
|
|
@ -250,7 +250,7 @@ func TestParse(t *testing.T) {
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
desc: "map struct with sub-map case senstitive",
|
desc: "map struct with sub-map case sensitive",
|
||||||
args: []string{"--foo.Name1.bar.name2.value=firstValue", "--foo.naMe1.bar.name2.value=secondValue"},
|
args: []string{"--foo.Name1.bar.name2.value=firstValue", "--foo.naMe1.bar.name2.value=secondValue"},
|
||||||
element: &struct {
|
element: &struct {
|
||||||
Foo map[string]struct {
|
Foo map[string]struct {
|
||||||
|
@ -273,6 +273,20 @@ func TestParse(t *testing.T) {
|
||||||
"traefik.foo.Name1.bar.name2.value": "secondValue",
|
"traefik.foo.Name1.bar.name2.value": "secondValue",
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
|
{
|
||||||
|
desc: "pointer of struct and map without explicit value",
|
||||||
|
args: []string{"--foo.default.bar.fuu"},
|
||||||
|
element: &struct {
|
||||||
|
Foo map[string]struct {
|
||||||
|
Bar *struct {
|
||||||
|
Fuu *struct{ Value string }
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}{},
|
||||||
|
expected: map[string]string{
|
||||||
|
"traefik.foo.default.bar.fuu": "true",
|
||||||
|
},
|
||||||
|
},
|
||||||
{
|
{
|
||||||
desc: "slice with several flags 2 and different cases.",
|
desc: "slice with several flags 2 and different cases.",
|
||||||
args: []string{"--foo", "bar", "--Foo", "baz"},
|
args: []string{"--foo", "bar", "--Foo", "baz"},
|
||||||
|
|
Loading…
Reference in a new issue