🔀 Merge pull request #1326 from rokiden/master

Added BACKUP_DIR env for conf.yml backups

.env

@@ -35,3 +35,6 @@
 
 # Again, set automatically using package.json during build time
 # VUE_APP_VERSION=2.0.0
+
+# Directory for conf.yml backups
+# BACKUP_DIR=./public/

docs/developing.md

@@ -80,6 +80,7 @@ You can set variables either in your environment, or using the [`.env`](https://
 - `INTEGRITY` - Should enable SRI for build script and link resources
 - `IS_DOCKER` - Computed automatically on build. Indicates if running in container
 - `VUE_APP_VERSION` - Again, set automatically using package.json during build time
+- `BACKUP_DIR` - Directory for conf.yml backups
 
 ### Environment Modes
 

services/save-config.js

@@ -5,6 +5,7 @@
  * Finally, it will call a function with the status message
  */
 const fsPromises = require('fs').promises;
+const path = require('path');
 
 module.exports = async (newConfig, render) => {
   /* Either returns nothing (if using default path), or strips navigational characters from path */
@@ -24,7 +25,8 @@ module.exports = async (newConfig, render) => {
   };
 
   // Make the full file name and path to save the backup config file
-  const backupFilePath = `${settings.defaultLocation}${usersFileName || settings.filename}-`
+  const backupFilePath = path.normalize(process.env.BACKUP_DIR || settings.defaultLocation) 
+    + `/${usersFileName || settings.filename}-`
     + `${Math.round(new Date() / 1000)}${settings.backupDenominator}`;
 
   // The path where the main conf.yml should be read and saved to