forks/go-ethereum
1
0
Fork 1
mirror of https://github.com/ethereum/go-ethereum.git synced 2026-07-17 18:30:45 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

internal/flags: add missing flag types for auto-env-var generation (#28692)

Browse source 
Certain flags, such as `--rpc.txfeecap` currently do not have an env-var auto-generated for them. This change adds three missing cli flag types to the auto env-var helper function to fix this.
This commit is contained in:
Daniel Liu 2024-11-25 16:39:29 +08:00
parent 4e192a5172
commit ac611170d1
1 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
internal/flags/helpers.go
View file

@ -94,7 +94,7 @@ func MigrateGlobalFlags(ctx *cli.Context) {
func doMigrateFlags(ctx *cli.Context) { func doMigrateFlags(ctx *cli.Context) {
// Figure out if there are any aliases of commands. If there are, we want // Figure out if there are any aliases of commands. If there are, we want
// to ignore them when iterating over the flags. // to ignore them when iterating over the flags.
var aliases = make(map[string]bool) aliases := make(map[string]bool)
for _, fl := range ctx.Command.Flags { for _, fl := range ctx.Command.Flags {
for _, alias := range fl.Names()[1:] { for _, alias := range fl.Names()[1:] {
aliases[alias] = true aliases[alias] = true
@ -228,15 +228,24 @@ func AutoEnvVars(flags []cli.Flag, prefix string) {
case *cli.StringFlag: case *cli.StringFlag:
flag.EnvVars = append(flag.EnvVars, envvar) flag.EnvVars = append(flag.EnvVars, envvar)
case *cli.StringSliceFlag:
flag.EnvVars = append(flag.EnvVars, envvar)
case *cli.BoolFlag: case *cli.BoolFlag:
flag.EnvVars = append(flag.EnvVars, envvar) flag.EnvVars = append(flag.EnvVars, envvar)
case *cli.IntFlag: case *cli.IntFlag:
flag.EnvVars = append(flag.EnvVars, envvar) flag.EnvVars = append(flag.EnvVars, envvar)
case *cli.Int64Flag:
flag.EnvVars = append(flag.EnvVars, envvar)
case *cli.Uint64Flag: case *cli.Uint64Flag:
flag.EnvVars = append(flag.EnvVars, envvar) flag.EnvVars = append(flag.EnvVars, envvar)
case *cli.Float64Flag:
flag.EnvVars = append(flag.EnvVars, envvar)
case *cli.DurationFlag: case *cli.DurationFlag:
flag.EnvVars = append(flag.EnvVars, envvar) flag.EnvVars = append(flag.EnvVars, envvar)