use atomic writes for settings

minbrowser · Dec 3, 2024 · be992c6 · be992c6
1 parent a5bb02d
commit be992c6
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/js/util/settings/settingsMain.js b/js/util/settings/settingsMain.js
@@ -1,3 +1,5 @@
+const writeFileAtomic = require('write-file-atomic')
+
 var settings = {
   filePath: null,
   fileWritePromise: null,
@@ -13,7 +15,15 @@ var settings = {
 
     /* eslint-disable no-inner-declarations */
     function newFileWrite () {
-      return fs.promises.writeFile(settings.filePath, JSON.stringify(settings.list))
+      return new Promise(function (resolve, reject) {
+        writeFileAtomic(settings.filePath, JSON.stringify(settings.list), {}, function (err) {
+          if (err) {
+            reject(err)
+          } else {
+            resolve()
+          }
+        })
+      })
     }
 
     function ongoingFileWrite () {