From 7bac5d5961125b3b2728c34632fd7b0fa2aff563 Mon Sep 17 00:00:00 2001 From: mike12345567 Date: Wed, 16 Feb 2022 12:36:24 +0000 Subject: [PATCH] Adding a check to make sure there is a tenant ID for the checklist call when looking up apps in multi-tenant envs. --- packages/worker/src/api/controllers/global/configs.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/packages/worker/src/api/controllers/global/configs.js b/packages/worker/src/api/controllers/global/configs.js index 0d3c8903f..2562dd4e2 100644 --- a/packages/worker/src/api/controllers/global/configs.js +++ b/packages/worker/src/api/controllers/global/configs.js @@ -246,12 +246,16 @@ exports.destroy = async function (ctx) { exports.configChecklist = async function (ctx) { const db = getGlobalDB() + const tenantId = getTenantId() try { // TODO: Watch get started video - // Apps exist - const apps = await getAllApps({ idsOnly: true, efficient: true }) + let apps = [] + if (!env.MULTI_TENANCY || tenantId) { + // Apps exist + apps = await getAllApps({ idsOnly: true, efficient: true }) + } // They have set up SMTP const smtpConfig = await getScopedFullConfig(db, {