From 74bdb6cb9d297cc5e7783963fb89e170b61a0ad1 Mon Sep 17 00:00:00 2001
From: Johannes Zellner <johannes@cloudron.io>
Date: Wed, 29 Jul 2020 19:50:08 +0200
Subject: [PATCH] Only mount app data volumes if localstorage is used

---
 src/sftp.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/sftp.js b/src/sftp.js
index 0295a2ce3..f2e11ad69 100644
--- a/src/sftp.js
+++ b/src/sftp.js
@@ -40,7 +40,7 @@ function rebuild(options, callback) {
 
         let dataDirs = [];
         result.forEach(function (app) {
-            if (!app.dataDir) return;
+            if (!app.manifest.addons['localstorage']) return;
 
             if (options.ignoredApps && options.ignoredApps.indexOf(app.id) !== -1) {
                 debug(`Ignoring volume for ${app.id}`);
@@ -49,7 +49,7 @@ function rebuild(options, callback) {
 
             dataDirs.push({
                 hostDir: app.dataDir || path.join(paths.APPS_DATA_DIR, app.id, 'data'),
-                // /data is required since this is where the localstorage data would be in APPS_DATA_DIR
+                // FIXME data/ prefix can be dropped if sftp addon code is changed
                 mountDir: `/app/data/${app.id}/data`
             });
         });