Merge pull request #3 from retrohacker/safe-write

Writes config to temp file then renames to dest
LinusU · Jul 22, 2016 · 14a70d2 · 14a70d2
2 parents 5311a5a + c05d039
commit 14a70d2
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -33,7 +33,15 @@ ApplicationConfig.prototype.write = function (data, cb) {
   mkdirp(directoryPath, function (err) {
     if (err) { return cb(err) }
 
-    fs.writeFile(self.filePath, JSON.stringify(data, null, 2), cb)
+    var tempFilePath =
+      self.filePath +
+      Math.random().toString().substr(2) +
+      Date.now().toString()
+    fs.writeFile(tempFilePath, JSON.stringify(data, null, 2), function (err) {
+      if (err) { return cb(err) }
+
+      fs.rename(tempFilePath, self.filePath, cb)
+    })
   })
 }