fix: don't update config field when writting fails

Signed-off-by: Norman Meier <norman@berty.tech>
berty · Nov 3, 2021 · 10b44c5 · 10b44c5
1 parent 4942bcd
commit 10b44c5
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/repo.go b/repo.go
@@ -71,14 +71,17 @@ func (r *encRepo) setConfig(updated *config.Config) error {
 
 	// Do not use `*r.config = ...`. This will modify the *shared* config
 	// returned by `r.Config`.
-	if r.config, err = config.FromMap(mapconf); err != nil {
+	conf, err := config.FromMap(mapconf)
+	if err != nil {
 		return err
 	}
 
-	if err := writeConfigToDatastore(r.root, mapconf); err != nil {
+	if err := writeConfigToDatastore(r.root, conf); err != nil {
 		return err
 	}
 
+	r.config = conf
+
 	return nil
 }