cli: Handle missing home directory
Some checks failed
DCO action / DCO (pull_request) Failing after 1m18s
Tests and linters / Run gofumpt (pull_request) Successful in 1m37s
Tests and linters / Staticcheck (pull_request) Successful in 3m7s
Tests and linters / Lint (pull_request) Successful in 3m35s
Pre-commit hooks / Pre-commit (pull_request) Successful in 3m44s
Build / Build Components (pull_request) Successful in 3m47s
Vulncheck / Vulncheck (pull_request) Successful in 3m39s
Tests and linters / gopls check (pull_request) Successful in 4m19s
Tests and linters / Tests with -race (pull_request) Successful in 5m2s
Tests and linters / Tests (pull_request) Successful in 5m59s
Some checks failed
DCO action / DCO (pull_request) Failing after 1m18s
Tests and linters / Run gofumpt (pull_request) Successful in 1m37s
Tests and linters / Staticcheck (pull_request) Successful in 3m7s
Tests and linters / Lint (pull_request) Successful in 3m35s
Pre-commit hooks / Pre-commit (pull_request) Successful in 3m44s
Build / Build Components (pull_request) Successful in 3m47s
Vulncheck / Vulncheck (pull_request) Successful in 3m39s
Tests and linters / gopls check (pull_request) Successful in 4m19s
Tests and linters / Tests with -race (pull_request) Successful in 5m2s
Tests and linters / Tests (pull_request) Successful in 5m59s
go-homedir library incorrectly handles some of the errors that could occur. It is archived, so no PR, but let's fix it on our side. The scenario in case: executing command in an empty environment. Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
parent
81f4cdbb91
commit
6649e006fc
1 changed files with 8 additions and 6 deletions
|
@ -114,12 +114,14 @@ func initConfig() {
|
|||
} else {
|
||||
// Find home directory.
|
||||
home, err := homedir.Dir()
|
||||
commonCmd.ExitOnErr(rootCmd, "", err)
|
||||
|
||||
// Search config in `$HOME/.config/frostfs-cli/` with name "config.yaml"
|
||||
viper.AddConfigPath(filepath.Join(home, ".config", "frostfs-cli"))
|
||||
viper.SetConfigName("config")
|
||||
viper.SetConfigType("yaml")
|
||||
if err != nil {
|
||||
common.PrintVerbose(rootCmd, "Get homedir: %s", err)
|
||||
} else {
|
||||
// Search config in `$HOME/.config/frostfs-cli/` with name "config.yaml"
|
||||
viper.AddConfigPath(filepath.Join(home, ".config", "frostfs-cli"))
|
||||
viper.SetConfigName("config")
|
||||
viper.SetConfigType("yaml")
|
||||
}
|
||||
}
|
||||
|
||||
viper.SetEnvPrefix(envPrefix)
|
||||
|
|
Loading…
Reference in a new issue