From e8b4ff02feade2886becdb1db92626ce9303dd5a Mon Sep 17 00:00:00 2001
From: bpizzi <baptiste@bpizzi.fr>
Date: Fri, 27 Jul 2018 10:10:14 +0200
Subject: [PATCH] Fixed missing f.Close() in writeConfig()

Defering can cause trouble because we're writing to the file outside the function
where the defering is registered, calling f.Sync() ensures that bytes are flushed
to disk even is Close() is called too soon.
---
 viper.go | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/viper.go b/viper.go
index 907a102..2631388 100644
--- a/viper.go
+++ b/viper.go
@@ -1266,7 +1266,13 @@ func (v *Viper) writeConfig(filename string, force bool) error {
 	if err != nil {
 		return err
 	}
-	return v.marshalWriter(f, configType)
+	defer f.Close()
+
+	if err := v.marshalWriter(f, configType); err != nil {
+		return err
+	}
+
+	return f.Sync()
 }
 
 // Unmarshal a Reader into a map.