STACKITGit/cli
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

add browser option to config

Browse source 
Allows setting the path to the browser using the config.

Closes #858
This commit is contained in:
Des Preston 2021-07-13 15:06:40 -04:00
parent 161de77fd7
commit c95f30af80
2 changed files with 20 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

13
pkg/cmd/factory/default.go
View file

@ -89,8 +89,21 @@ func httpClientFunc(f *cmdutil.Factory, appVersion string) func() (*http.Client,
}
}
// browser returns the path to the browser. Attempts to read a path from Config,
// and falls back to the $BROWSER env var if Config value is blank.
func browser(f *cmdutil.Factory) cmdutil.Browser {
io := f.IOStreams
cfg, err := f.Config()
if err != nil {
fmt.Fprintf(io.ErrOut, "problem loading config %s", err)
}
fromConfig, err := cfg.Get("", "browser")
if err == nil && fromConfig != "" {
return cmdutil.NewBrowser(fromConfig, io.Out, io.ErrOut)
}
return cmdutil.NewBrowser(os.Getenv("BROWSER"), io.Out, io.ErrOut)
}