config file can't be optional

[joshuarubin]

It would be more useful if ReadInConfig could return an error instead of causing the process to Exit if the config can't be found. This would enable the case where the whole file can be optional and the values read from defaults.

func ReadInConfig() {
    jww.INFO.Println("Attempting to read in config file")

    if !stringInSlice(getConfigType(), SupportedExts) {
        jww.ERROR.Fatalf("Unsupported Config Type %q", getConfigType())
    }

    ...

[spf13]

Agree. Please Send me a pull request.

Steve Francia
spf13.com
@spf13

On Apr 8, 2014, at 4:18 AM, Joshua Rubin [email protected] wrote:

It would be more useful if ReadInConfig could return an error instead of causing the process to Exit if the config can't be found. This would enable the case where the whole file can be optional and the values read from defaults.

func ReadInConfig() {
jww.INFO.Println("Attempting to read in config file")

if !stringInSlice(getConfigType(), SupportedExts) {
    jww.ERROR.Fatalf("Unsupported Config Type %q", getConfigType())
}

...

—
Reply to this email directly or view it on GitHub.

[joshuarubin]

Closed in favor of #2

[stevenroose]

So it is really not possible to have Viper not error when the config file does not exist?

[balta2ar]

So it is really not possible to have Viper not error when the config file does not exist?

I have the same question... @spf13 @joshuarubin?

[sagikazarmark]

Leaving the solution here for future reference:

	err := viper.ReadInConfig()
	if _, ok := err.(viper.ConfigFileNotFoundError); !ok {
		panic(err)
	}

Basically you just have to check the error and ignore it if it is a config not found error.

[alex-shpak]

If somebody interested, alternative solution It's not the same

if err := config.ReadConfig(strings.NewReader("")); err != nil {
	log.Fatal(err)
}

[sagikazarmark]

Hm, calling ReadConfig is totally optional, you only need it if you want to read a config file, so I'm failing to see how this helps.

[alex-shpak]

Oh, yeah, you are right.
Scratch that.