diff --git a/packages/auth/src/db/utils.js b/packages/auth/src/db/utils.js index 100dc005c8..1fc78f4182 100644 --- a/packages/auth/src/db/utils.js +++ b/packages/auth/src/db/utils.js @@ -1,5 +1,8 @@ const { newid } = require("../hashing") const Replication = require("./Replication") +const env = require("../environment") +const fetch = require("node-fetch") +const { getCouch } = require("./index") const UNICODE_MAX = "\ufff0" const SEPARATOR = "_" @@ -156,6 +159,23 @@ exports.getDeployedAppID = appId => { return appId } +/** + * if in production this will use the CouchDB _all_dbs call to retrieve a list of databases. If testing + * when using Pouch it will use the pouchdb-all-dbs package. + */ +exports.getAllDbs = async () => { + // specifically for testing we use the pouch package for this + if (env.isTest()) { + return getCouch().allDbs() + } + const response = await fetch(`${env.COUCH_DB_URL}/_all_dbs`) + if (response.status === 200) { + return response.json() + } else { + throw "Cannot connect to CouchDB instance" + } +} + /** * Lots of different points in the system need to find the full list of apps, this will * enumerate the entire CouchDB cluster and get the list of databases (every app). @@ -164,10 +184,8 @@ exports.getDeployedAppID = appId => { * @return {Promise} returns the app information document stored in each app database. */ exports.getAllApps = async ({ CouchDB, dev, all } = {}) => { - let allDbs = await CouchDB.allDbs() - const appDbNames = allDbs.filter(dbName => - dbName.startsWith(exports.APP_PREFIX) - ) + let dbs = await exports.getAllDbs() + const appDbNames = dbs.filter(dbName => dbName.startsWith(exports.APP_PREFIX)) const appPromises = appDbNames.map(db => // skip setup otherwise databases could be re-created new CouchDB(db, { skip_setup: true }).get(DocumentTypes.APP_METADATA) diff --git a/packages/auth/src/middleware/passport/tests/utilities/db.js b/packages/auth/src/middleware/passport/tests/utilities/db.js index 3a792e3272..e83784471b 100644 --- a/packages/auth/src/middleware/passport/tests/utilities/db.js +++ b/packages/auth/src/middleware/passport/tests/utilities/db.js @@ -1,5 +1,4 @@ const PouchDB = require("pouchdb") -const allDbs = require("pouchdb-all-dbs") const env = require("../../../../environment") let POUCH_DB_DEFAULTS @@ -15,6 +14,4 @@ if (env.isTest()) { const Pouch = PouchDB.defaults(POUCH_DB_DEFAULTS) -allDbs(Pouch) - module.exports = Pouch diff --git a/packages/builder/src/components/start/CreateAppModal.svelte b/packages/builder/src/components/start/CreateAppModal.svelte index 8d2ec56655..93eeec77bb 100644 --- a/packages/builder/src/components/start/CreateAppModal.svelte +++ b/packages/builder/src/components/start/CreateAppModal.svelte @@ -9,6 +9,7 @@ Checkbox, } from "@budibase/bbui" import { store, automationStore, hostingStore } from "builderStore" + import { admin } from "stores/portal" import { string, mixed, object } from "yup" import api, { get, post } from "builderStore/api" import analytics from "analytics" @@ -102,6 +103,8 @@ if (applicationPkg.ok) { await store.actions.initialise(pkg) await automationStore.actions.fetch() + // update checklist - incase first app + await admin.init() } else { throw new Error(pkg) } diff --git a/packages/builder/src/pages/builder/portal/apps/index.svelte b/packages/builder/src/pages/builder/portal/apps/index.svelte index aafe9b60c3..50b49de9da 100644 --- a/packages/builder/src/pages/builder/portal/apps/index.svelte +++ b/packages/builder/src/pages/builder/portal/apps/index.svelte @@ -18,7 +18,7 @@ import api, { del } from "builderStore/api" import analytics from "analytics" import { onMount } from "svelte" - import { apps, auth } from "stores/portal" + import { apps, auth, admin } from "stores/portal" import download from "downloadjs" import { goto } from "@roxi/routify" import ConfirmDialog from "components/common/ConfirmDialog.svelte" @@ -159,6 +159,8 @@ throw json.message } await apps.load() + // get checklist, just in case that was the last app + await admin.init() notifications.success("App deleted successfully") } catch (err) { notifications.error(`Error deleting app: ${err}`) diff --git a/packages/builder/src/stores/portal/organisation.js b/packages/builder/src/stores/portal/organisation.js index 71c0be4b4d..df950b92a0 100644 --- a/packages/builder/src/stores/portal/organisation.js +++ b/packages/builder/src/stores/portal/organisation.js @@ -2,7 +2,7 @@ import { writable, get } from "svelte/store" import api from "builderStore/api" const DEFAULT_CONFIG = { - platformUrl: "http://localhost:1000", + platformUrl: "http://localhost:10000", logoUrl: undefined, docsUrl: undefined, company: "Budibase", diff --git a/packages/server/scripts/replicateApp.js b/packages/server/scripts/replicateApp.js index a27f77b9a0..8b363309ab 100644 --- a/packages/server/scripts/replicateApp.js +++ b/packages/server/scripts/replicateApp.js @@ -7,6 +7,7 @@ const CouchDB = require("../src/db") const { DocumentTypes } = require("../src/db/utils") +const { getAllDbs } = require("@budibase/auth/db") const appName = process.argv[2].toLowerCase() const remoteUrl = process.argv[3] @@ -14,8 +15,8 @@ const remoteUrl = process.argv[3] console.log(`Replicating from ${appName} to ${remoteUrl}/${appName}`) const run = async () => { - const allDbs = await CouchDB.allDbs() - const appDbNames = allDbs.filter(dbName => dbName.startsWith("inst_app")) + const dbs = await getAllDbs() + const appDbNames = dbs.filter(dbName => dbName.startsWith("inst_app")) let apps = [] for (let dbName of appDbNames) { const db = new CouchDB(dbName) diff --git a/packages/server/src/db/client.js b/packages/server/src/db/client.js index fbc25986d5..d50d72b18d 100644 --- a/packages/server/src/db/client.js +++ b/packages/server/src/db/client.js @@ -24,21 +24,7 @@ if (env.isTest()) { const Pouch = PouchDB.defaults(POUCH_DB_DEFAULTS) +// have to still have pouch alldbs for testing allDbs(Pouch) -// replicate your local levelDB pouch to a running HTTP compliant couch or pouchdb server. -/* istanbul ignore next */ -// eslint-disable-next-line no-unused-vars -function replicateLocal() { - Pouch.allDbs().then(dbs => { - for (let db of dbs) { - new Pouch(db).sync( - new PouchDB(`http://127.0.0.1:5984/${db}`, { live: true }) - ) - } - }) -} - -// replicateLocal() - module.exports = Pouch diff --git a/packages/worker/src/api/controllers/admin/configs.js b/packages/worker/src/api/controllers/admin/configs.js index 78caa817b2..0b90d8fb5a 100644 --- a/packages/worker/src/api/controllers/admin/configs.js +++ b/packages/worker/src/api/controllers/admin/configs.js @@ -5,6 +5,7 @@ const { getConfigParams, getGlobalUserParams, getScopedFullConfig, + getAllDbs, } = require("@budibase/auth").db const { Configs } = require("../../../constants") const email = require("../../../utilities/email") @@ -229,8 +230,8 @@ exports.configChecklist = async function (ctx) { // TODO: Watch get started video // Apps exist - let allDbs = await CouchDB.allDbs() - const appDbNames = allDbs.filter(dbName => dbName.startsWith(APP_PREFIX)) + let dbs = await getAllDbs() + const appDbNames = dbs.filter(dbName => dbName.startsWith(APP_PREFIX)) // They have set up SMTP const smtpConfig = await getScopedFullConfig(db, { diff --git a/packages/worker/src/api/controllers/app.js b/packages/worker/src/api/controllers/app.js index ff9692a5ec..e3dbaf5f1a 100644 --- a/packages/worker/src/api/controllers/app.js +++ b/packages/worker/src/api/controllers/app.js @@ -1,15 +1,14 @@ -const { DocumentTypes } = require("@budibase/auth").db +const { DocumentTypes, getAllDbs } = require("@budibase/auth").db const CouchDB = require("../../db") const APP_PREFIX = "app_" const URL_REGEX_SLASH = /\/|\\/g exports.getApps = async ctx => { - // allDbs call of CouchDB is very inaccurate in production - const allDbs = await CouchDB.allDbs() - const appDbNames = allDbs.filter(dbName => dbName.startsWith(APP_PREFIX)) + const dbs = await getAllDbs() + const appDbNames = dbs.filter(dbName => dbName.startsWith(APP_PREFIX)) const appPromises = appDbNames.map(db => - new CouchDB(db).get(DocumentTypes.APP_METADATA) + new CouchDB(db, { skip_setup: true }).get(DocumentTypes.APP_METADATA) ) const apps = await Promise.allSettled(appPromises) diff --git a/packages/worker/src/db/index.js b/packages/worker/src/db/index.js index 1ade1cd6db..d6d035cff1 100644 --- a/packages/worker/src/db/index.js +++ b/packages/worker/src/db/index.js @@ -19,6 +19,7 @@ if (env.isTest()) { const Pouch = PouchDB.defaults(POUCH_DB_DEFAULTS) +// have to still have pouch alldbs for testing allDbs(Pouch) module.exports = Pouch