redis: make optional

part of #810

src/apps.js

@@ -43,6 +43,7 @@ exports = module.exports = {
     setMailbox,
     setInbox,
     setTurn,
+    setRedis,
     setLocation,
     setStorage,
     repair,
@@ -200,7 +201,7 @@ const APPS_FIELDS_PREFIXED = [ 'apps.id', 'apps.appStoreId', 'apps.installationS
     'apps.sso', 'apps.debugModeJson', 'apps.enableBackup', 'apps.proxyAuth', 'apps.containerIp', 'apps.crontab',
     'apps.creationTime', 'apps.updateTime', 'apps.enableAutomaticUpdate', 'apps.upstreamUri',
     'apps.enableMailbox', 'apps.mailboxDisplayName', 'apps.mailboxName', 'apps.mailboxDomain', 'apps.enableInbox', 'apps.inboxName', 'apps.inboxDomain',
-    'apps.enableTurn', 'apps.storageVolumeId', 'apps.storageVolumePrefix', 'apps.ts', 'apps.healthTime', '(apps.icon IS NOT NULL) AS hasIcon', '(apps.appStoreIcon IS NOT NULL) AS hasAppStoreIcon' ].join(',');
+    'apps.enableTurn', 'apps.enableRedis', 'apps.storageVolumeId', 'apps.storageVolumePrefix', 'apps.ts', 'apps.healthTime', '(apps.icon IS NOT NULL) AS hasIcon', '(apps.appStoreIcon IS NOT NULL) AS hasAppStoreIcon' ].join(',');
 
 // const PORT_BINDINGS_FIELDS = [ 'hostPort', 'type', 'environmentVariable', 'appId' ].join(',');
 const LOCATION_FIELDS = [ 'appId', 'subdomain', 'domain', 'type', 'certificateJson' ];
@@ -595,7 +596,7 @@ function removeInternalFields(app) {
         'accessRestriction', 'manifest', 'portBindings', 'iconUrl', 'memoryLimit', 'cpuShares', 'operators',
         'sso', 'debugMode', 'reverseProxyConfig', 'enableBackup', 'creationTime', 'updateTime', 'ts', 'tags',
         'label', 'secondaryDomains', 'redirectDomains', 'aliasDomains', 'env', 'enableAutomaticUpdate',
-        'storageVolumeId', 'storageVolumePrefix', 'mounts', 'enableTurn',
+        'storageVolumeId', 'storageVolumePrefix', 'mounts', 'enableTurn', 'enableRedis',
         'enableMailbox', 'mailboxDisplayName', 'mailboxName', 'mailboxDomain', 'enableInbox', 'inboxName', 'inboxDomain');
 
     removeCertificateKeys(result);
@@ -843,6 +844,7 @@ async function add(id, appStoreId, manifest, subdomain, domain, portBindings, da
         enableMailbox = data.enableMailbox || false,
         upstreamUri = data.upstreamUri || '',
         enableTurn = 'enableTurn' in data ? data.enableTurn : true,
+        enableRedis = 'enableRedis' in data ? data.enableRedis : true,
         icon = data.icon || null;
 
     const queries = [];
@@ -850,11 +852,11 @@ async function add(id, appStoreId, manifest, subdomain, domain, portBindings, da
     queries.push({
         query: 'INSERT INTO apps (id, appStoreId, manifestJson, installationState, runState, accessRestrictionJson, memoryLimit, cpuShares, '
             + 'sso, debugModeJson, mailboxName, mailboxDomain, label, tagsJson, reverseProxyConfigJson, servicesConfigJson, icon, '
-            + 'enableMailbox, mailboxDisplayName, upstreamUri, enableTurn) '
-            + ' VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)',
+            + 'enableMailbox, mailboxDisplayName, upstreamUri, enableTurn, enableRedis) '
+            + ' VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)',
         args: [ id, appStoreId, manifestJson, installationState, runState, accessRestrictionJson, memoryLimit, cpuShares,
             sso, debugModeJson, mailboxName, mailboxDomain, label, tagsJson, reverseProxyConfigJson, servicesConfigJson, icon,
-            enableMailbox, mailboxDisplayName, upstreamUri, enableTurn ]
+            enableMailbox, mailboxDisplayName, upstreamUri, enableTurn, enableRedis ]
     });
 
     queries.push({
@@ -1328,6 +1330,7 @@ async function install(data, auditSource) {
         overwriteDns = 'overwriteDns' in data ? data.overwriteDns : false,
         skipDnsSetup = 'skipDnsSetup' in data ? data.skipDnsSetup : false,
         enableTurn = 'enableTurn' in data ? data.enableTurn : true,
+        enableRedis = 'enableRedis' in data ? data.enableRedis : true,
         appStoreId = data.appStoreId,
         upstreamUri = data.upstreamUri || '',
         manifest = data.manifest;
@@ -1416,6 +1419,7 @@ async function install(data, auditSource) {
         enableMailbox,
         upstreamUri,
         enableTurn,
+        enableRedis,
         runState: exports.RSTATE_RUNNING,
         installationState: exports.ISTATE_PENDING_INSTALL
     };
@@ -1761,6 +1765,29 @@ async function setTurn(app, enableTurn, auditSource) {
     return { taskId };
 }
 
+async function setRedis(app, enableRedis, auditSource) {
+    assert.strictEqual(typeof app, 'object');
+    assert.strictEqual(typeof enableRedis, 'boolean');
+    assert.strictEqual(typeof auditSource, 'object');
+
+    const appId = app.id;
+    let error = checkAppState(app, exports.ISTATE_PENDING_SERVICES_CHANGE);
+    if (error) throw error;
+
+    if (!app.manifest.addons?.redis) throw new BoxError(BoxError.BAD_FIELD, 'App does not use redis addon');
+    if (!app.manifest.addons.redis.optional) throw new BoxError(BoxError.BAD_FIELD, 'redis service is not optional');
+
+    const task = {
+        args: {},
+        values: { enableRedis }
+    };
+    const taskId = await addTask(appId, exports.ISTATE_PENDING_SERVICES_CHANGE, task, auditSource);
+
+    await eventlog.add(eventlog.ACTION_APP_CONFIGURE, auditSource, { appId, app, enableRedis, taskId });
+
+    return { taskId };
+}
+
 async function setAutomaticBackup(app, enable, auditSource) {
     assert.strictEqual(typeof app, 'object');
     assert.strictEqual(typeof enable, 'boolean');