Ensure the storage location resides in the Docker volume when required

Dockerfile

@@ -19,6 +19,7 @@ ENV NODE_CONFIG_DIR=/data/config
 ENV BOT_PORT=4501
 ENV BOT_BIND=0.0.0.0
 ENV BOT_DATABASE=/data/trello.db
+ENV BOT_DATA_PATH=/data/storage
 ENV BOT_DOCKER_LOGS=true
 
 CMD node /matrix-trello-bot/index.js

config/default.yaml

@@ -14,6 +14,9 @@ publicBaseUrl: "https://your.domain.com"
 # Where the database is located. Not used by the Docker image, and instead relies on the volume.
 dbFile: "trello.db"
 
+# Where sync data and other related information should be stored
+dataPath: "./storage"
+
 # Settings for controlling how logging works
 # The Docker image will only consider the consoleLevel, and not write to any files.
 logging:

src/config.ts

@@ -6,6 +6,7 @@ interface IConfig {
     accessToken: string;
 
     dbFile: string;
+    dataPath: string;
 
     trelloApiKey: string;
     trelloApiSecret: string;
@@ -46,6 +47,15 @@ if (process.env["BOT_DATABASE"]) {
     conf.dbFile = readlDbPath;
 }
 
+if (process.env["BOT_DATA_PATH"]) {
+    const realPath = process.env["BOT_DATA_PATH"];
+    if (realPath !== conf.dataPath) {
+        console.warn("Configuration and environment variables do not agree on the data path. Using " + realPath);
+    }
+
+    conf.dataPath = realPath;
+}
+
 if (process.env["BOT_DOCKER_LOGS"]) {
     console.log("Altering log configuration to only write out to console");
     conf.logging = {

src/index.ts

@@ -7,7 +7,7 @@ import { CommandProcessor } from "./CommandProcessor";
 import { WebhookProcessor } from "./notifications/WebhookProcessor";
 
 LogService.configure(config.logging);
-const storageProvider = new LocalstorageStorageProvider("./storage");
+const storageProvider = new LocalstorageStorageProvider(config.dataPath);
 const client = new MatrixClient(config.homeserverUrl, config.accessToken, storageProvider);
 const commands = new CommandProcessor(client);
 const processor = new WebhookProcessor(client);