Always return valid Config and check for directory. Fixes #3

client/client.go

@@ -37,7 +37,7 @@ func NewDefaultConfigRetreiver(next ConfigRetriever) *DefaultConfigRetreiver {
 // it. If the config is empty a default config is returned.
 func (c *DefaultConfigRetreiver) RetrieveConfig(ctx context.Context, name string) (*Config, error) {
 	config, err := c.next.RetrieveConfig(ctx, name)
-	if err == nil && (config == nil || config.BaseURL == "") {
+	if config == nil || config.BaseURL == "" {
 		config = &Config{BaseURL: DefaultBaseURL}
 	}
 	return config, err

storage/file/file.go

@@ -23,8 +23,18 @@ type FileStorage struct {
 // New creates a new FileStorage backend.
 func New(path string) (*FileStorage, error) {
 	err := os.Mkdir(path, 0755)
-	if err != nil && !errors.Is(err, os.ErrExist) {
-		return nil, err
+	if err != nil {
+		if errors.Is(err, os.ErrExist) {
+			f, err := os.Stat(path)
+			if err != nil {
+				return nil, err
+			}
+			if !f.IsDir() {
+				return nil, errors.New("path is not a directory")
+			}
+		} else {
+			return nil, err
+		}
 	}
 	return &FileStorage{path: path}, nil
 }