making checklist cache length controllable through env var

packages/worker/src/api/controllers/global/configs.js

@@ -17,7 +17,6 @@ const { googleCallbackUrl, oidcCallbackUrl } = require("./auth")
 const {
   withCache,
   CacheKeys,
-  TTL,
   bustCache,
 } = require("@budibase/backend-core/cache")
 
@@ -256,58 +255,62 @@ exports.configChecklist = async function (ctx) {
   const tenantId = getTenantId()
 
   try {
-    ctx.body = await withCache(CacheKeys.CHECKLIST, TTL.ONE_HOUR, async () => {
+    ctx.body = await withCache(
-      let apps = []
+      CacheKeys.CHECKLIST,
-      if (!env.MULTI_TENANCY || tenantId) {
+      env.CHECKLIST_CACHE_TTL,
-        // Apps exist
+      async () => {
-        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, {
+          type: Configs.SMTP,
+        })
+
+        // They have set up Google Auth
+        const googleConfig = await getScopedFullConfig(db, {
+          type: Configs.GOOGLE,
+        })
 
-      // They have set up SMTP
+        // They have set up OIDC
-      const smtpConfig = await getScopedFullConfig(db, {
+        const oidcConfig = await getScopedFullConfig(db, {
-        type: Configs.SMTP,
+          type: Configs.OIDC,
-      })
-
-      // They have set up Google Auth
-      const googleConfig = await getScopedFullConfig(db, {
-        type: Configs.GOOGLE,
-      })
-
-      // They have set up OIDC
-      const oidcConfig = await getScopedFullConfig(db, {
-        type: Configs.OIDC,
-      })
-
-      // They have set up an global user
-      const users = await db.allDocs(
-        getGlobalUserParams(null, {
-          include_docs: true,
-          limit: 1,
         })
-      )
+
-      return {
+        // They have set up an global user
-        apps: {
+        const users = await db.allDocs(
-          checked: apps.length > 0,
+          getGlobalUserParams(null, {
-          label: "Create your first app",
+            include_docs: true,
-          link: "/builder/portal/apps",
+            limit: 1,
-        },
+          })
-        smtp: {
+        )
-          checked: !!smtpConfig,
+        return {
-          label: "Set up email",
+          apps: {
-          link: "/builder/portal/manage/email",
+            checked: apps.length > 0,
-        },
+            label: "Create your first app",
-        adminUser: {
+            link: "/builder/portal/apps",
-          checked: users && users.rows.length >= 1,
+          },
-          label: "Create your first user",
+          smtp: {
-          link: "/builder/portal/manage/users",
+            checked: !!smtpConfig,
-        },
+            label: "Set up email",
-        sso: {
+            link: "/builder/portal/manage/email",
-          checked: !!googleConfig || !!oidcConfig,
+          },
-          label: "Set up single sign-on",
+          adminUser: {
-          link: "/builder/portal/manage/auth",
+            checked: users && users.rows.length >= 1,
-        },
+            label: "Create your first user",
+            link: "/builder/portal/manage/users",
+          },
+          sso: {
+            checked: !!googleConfig || !!oidcConfig,
+            label: "Set up single sign-on",
+            link: "/builder/portal/manage/auth",
+          },
+        }
       }
-    })
+    )
   } catch (err) {
     ctx.throw(err.status, err)
   }

packages/worker/src/environment.js

@@ -1,4 +1,5 @@
 const { join } = require("path")
+const { TTL } = require("@budibase/backend-core/cache")
 
 function isDev() {
   return process.env.NODE_ENV !== "production"
@@ -52,7 +53,8 @@ module.exports = {
   SMTP_FROM_ADDRESS: process.env.SMTP_FROM_ADDRESS,
   PLATFORM_URL: process.env.PLATFORM_URL,
   COOKIE_DOMAIN: process.env.COOKIE_DOMAIN,
-  CHECKLIST_CACHE_TTL: parseIntSafe(process.env.CHECKLIST_CACHE_TTL) || 600,
+  CHECKLIST_CACHE_TTL:
+    parseIntSafe(process.env.CHECKLIST_CACHE_TTL) || TTL.ONE_HOUR,
   APPS_URL: process.env.APPS_URL,
   _set(key, value) {
     process.env[key] = value