TrueCloudLab/frostfs-node
20
1
Fork 28
Code Issues 107 Pull requests 4 Projects Releases 2 Packages 4 Activity Actions

[#532] cli: Respect XDG base directory spec
All checks were successful
DCO action / DCO (pull_request) Successful in 3m20s
Details
Tests and linters / Staticcheck (pull_request) Successful in 4m29s
Details
Tests and linters / gopls check (pull_request) Successful in 5m19s
Details
Tests and linters / Run gofumpt (pull_request) Successful in 7m21s
Details
Pre-commit hooks / Pre-commit (pull_request) Successful in 7m30s
Details
Build / Build Components (pull_request) Successful in 7m46s
Details
Tests and linters / Lint (pull_request) Successful in 8m41s
Details
Tests and linters / Tests (pull_request) Successful in 9m33s
Details
Tests and linters / Tests with -race (pull_request) Successful in 9m49s
Details
Vulncheck / Vulncheck (pull_request) Successful in 1m16s
Details

Browse source 
XDG base directory specification defines where various files
should be looked by an application. Hopefully, this makes `frostfs-cli`
more predictable and pleasant to work with. Luckily for us, golang already
has everything we need in the stdlib. This commit also gets rid of
`github.com/mitchellh/go-homedir` dependency.

Close #532
Refs #1455

Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
Evgenii Stratonikov 2024-11-05 16:15:59 +03:00
parent 6c45a17af6
commit 17ec84151b
Signed by: fyrchik
SSH key fingerprint: SHA256:m/TTwCzjnRkXgnzEx9X92ccxy1CcVeinOgDb3NPWWmg
3 changed files with 5 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

11
cmd/frostfs-cli/modules/root.go
View file

@ -21,7 +21,6 @@ import (
"git.frostfs.info/TrueCloudLab/frostfs-node/misc"
"git.frostfs.info/TrueCloudLab/frostfs-node/pkg/util/config"
"git.frostfs.info/TrueCloudLab/frostfs-node/pkg/util/gendoc"
"github.com/mitchellh/go-homedir"
"github.com/spf13/cobra"
"github.com/spf13/viper"
)
@ -112,13 +111,13 @@ func initConfig() {
// Use config file from the flag.
viper.SetConfigFile(cfgFile)
} else {
// Find home directory.
home, err := homedir.Dir()
// Find config directory.
configDir, err := os.UserConfigDir()
if err != nil {
common.PrintVerbose(rootCmd, "Get homedir: %s", err)
common.PrintVerbose(rootCmd, "Get config dir: %s", err)
} else {
// Search config in `$HOME/.config/frostfs-cli/` with name "config.yaml"
viper.AddConfigPath(filepath.Join(home, ".config", "frostfs-cli"))
// Search config in `$XDG_CONFIG_HOME/frostfs-cli/` with name "config.yaml"
viper.AddConfigPath(filepath.Join(configDir, "frostfs-cli"))
viper.SetConfigName("config")
viper.SetConfigType("yaml")
}