@budibase/server 1.0.94 → 1.0.97

package/coverage/lcov.info

@@ -1,55 +1,52 @@
 TN:
 SF:src/api/index.js
-FN:30,(anonymous_0)
-FN:37,(anonymous_1)
-FN:38,(anonymous_2)
-FN:60,(anonymous_3)
-FN:77,(anonymous_4)
-FNF:5
+FN:16,(anonymous_0)
+FN:17,(anonymous_1)
+FN:32,(anonymous_2)
+FN:58,(anonymous_3)
+FNF:4
 FNH:4
-FNDA:190,(anonymous_0)
+FNDA:1,(anonymous_0)
 FNDA:1,(anonymous_1)
-FNDA:1,(anonymous_2)
+FNDA:188,(anonymous_2)
 FNDA:186,(anonymous_3)
-FNDA:0,(anonymous_4)
 DA:1,24
+DA:6,24
 DA:7,24
 DA:8,24
 DA:9,24
 DA:10,24
 DA:11,24
 DA:12,24
-DA:13,24
-DA:15,24
+DA:14,24
+DA:16,24
 DA:17,24
-DA:31,190
-DA:35,190
-DA:37,1
-DA:38,1
-DA:60,24
-DA:61,186
-DA:62,186
-DA:64,39
-DA:65,39
-DA:70,39
-DA:71,0
-DA:72,0
-DA:77,24
-DA:80,24
-DA:81,624
-DA:82,624
+DA:19,24
+DA:33,188
+DA:37,188
+DA:58,24
+DA:59,186
+DA:60,186
+DA:62,39
+DA:63,39
+DA:68,39
+DA:69,0
+DA:70,0
+DA:76,24
+DA:77,624
+DA:78,624
+DA:81,24
+DA:82,24
 DA:85,24
 DA:86,24
-DA:89,24
-DA:90,24
-DA:92,24
-LF:31
-LH:29
-BRDA:64,0,0,39
-BRDA:64,0,1,0
-BRDA:64,0,2,0
-BRDA:70,1,0,0
-BRDA:70,1,1,39
+DA:88,24
+LF:30
+LH:28
+BRDA:62,0,0,39
+BRDA:62,0,1,0
+BRDA:62,0,2,0
+BRDA:68,1,0,0
+BRDA:68,1,1,39
 BRF:5
 BRH:2
 end_of_record
@@ -2422,8 +2419,8 @@ FN:141,(anonymous_11)
 FNF:12
 FNH:11
 FNDA:119,pickApi
-FNDA:121,getTableId
-FNDA:6,(anonymous_2)
+FNDA:122,getTableId
+FNDA:7,(anonymous_2)
 FNDA:74,(anonymous_3)
 FNDA:7,(anonymous_4)
 FNDA:4,(anonymous_5)
@@ -2439,37 +2436,37 @@ DA:3,40
 DA:6,119
 DA:7,0
 DA:9,118
-DA:13,121
-DA:14,77
-DA:16,44
-DA:17,36
+DA:13,122
+DA:14,80
+DA:16,42
+DA:17,34
 DA:19,8
 DA:20,7
 DA:24,40
-DA:25,6
-DA:26,6
-DA:27,6
-DA:29,6
+DA:25,7
+DA:26,7
+DA:27,7
+DA:29,7
 DA:30,0
-DA:32,6
-DA:33,6
-DA:34,4
-DA:35,4
-DA:37,4
-DA:38,4
+DA:32,7
+DA:33,7
+DA:34,5
+DA:35,5
+DA:37,5
+DA:38,5
 DA:40,2
 DA:44,40
 DA:45,74
 DA:46,74
 DA:47,74
 DA:49,74
-DA:50,2
-DA:52,72
-DA:53,72
-DA:54,71
-DA:55,71
-DA:56,71
-DA:57,71
+DA:50,3
+DA:52,71
+DA:53,71
+DA:54,70
+DA:55,70
+DA:56,70
+DA:57,70
 DA:59,1
 DA:63,40
 DA:64,7
@@ -2527,30 +2524,30 @@ LF:90
 LH:80
 BRDA:6,0,0,0
 BRDA:6,0,1,118
-BRDA:13,1,0,77
-BRDA:13,1,1,44
-BRDA:13,2,0,121
-BRDA:13,2,1,108
-BRDA:16,3,0,36
+BRDA:13,1,0,80
+BRDA:13,1,1,42
+BRDA:13,2,0,122
+BRDA:13,2,1,109
+BRDA:16,3,0,34
 BRDA:16,3,1,8
-BRDA:16,4,0,44
-BRDA:16,4,1,44
+BRDA:16,4,0,42
+BRDA:16,4,1,42
 BRDA:19,5,0,7
 BRDA:19,5,1,1
 BRDA:19,6,0,8
 BRDA:19,6,1,8
 BRDA:29,7,0,0
-BRDA:29,7,1,6
-BRDA:29,8,0,6
-BRDA:29,8,1,6
-BRDA:35,9,0,4
-BRDA:35,9,1,3
-BRDA:49,10,0,2
-BRDA:49,10,1,72
+BRDA:29,7,1,7
+BRDA:29,8,0,7
+BRDA:29,8,1,7
+BRDA:35,9,0,5
+BRDA:35,9,1,4
+BRDA:49,10,0,3
+BRDA:49,10,1,71
 BRDA:49,11,0,74
 BRDA:49,11,1,74
-BRDA:55,12,0,71
-BRDA:55,12,1,11
+BRDA:55,12,0,70
+BRDA:55,12,1,10
 BRDA:95,13,0,2
 BRDA:95,13,1,4
 BRDA:99,14,0,3
@@ -2595,8 +2592,8 @@ FNH:22
 FNDA:6,getView
 FNDA:5,getRawTableData
 FNDA:10,(anonymous_2)
-FNDA:5,(anonymous_3)
-FNDA:72,(anonymous_4)
+FNDA:6,(anonymous_3)
+FNDA:71,(anonymous_4)
 FNDA:7,(anonymous_5)
 FNDA:3,(anonymous_6)
 FNDA:3,(anonymous_7)
@@ -2657,44 +2654,44 @@ DA:71,5
 DA:76,10
 DA:78,5
 DA:81,40
-DA:82,5
-DA:83,5
-DA:84,5
-DA:85,5
-DA:87,5
-DA:88,5
+DA:82,6
+DA:83,6
+DA:84,6
+DA:85,6
+DA:87,6
+DA:88,6
 DA:90,0
 DA:93,0
 DA:97,0
-DA:100,5
-DA:102,5
-DA:103,5
-DA:104,21
-DA:105,6
-DA:109,5
-DA:110,5
-DA:115,5
+DA:100,6
+DA:102,6
+DA:103,6
+DA:104,26
+DA:105,7
+DA:109,6
+DA:110,6
+DA:115,6
 DA:116,1
-DA:120,4
-DA:127,4
-DA:129,4
+DA:120,5
+DA:127,5
+DA:129,5
 DA:131,0
 DA:132,0
 DA:133,0
-DA:136,4
+DA:136,5
 DA:142,40
-DA:143,72
-DA:144,72
-DA:145,72
-DA:147,72
-DA:148,72
-DA:152,72
-DA:153,72
-DA:154,72
-DA:159,72
+DA:143,71
+DA:144,71
+DA:145,71
+DA:147,71
+DA:148,71
+DA:152,71
+DA:153,71
+DA:154,71
+DA:159,71
 DA:160,0
-DA:164,72
-DA:171,71
+DA:164,71
+DA:171,70
 DA:177,40
 DA:178,7
 DA:181,7
@@ -2846,18 +2843,18 @@ BRDA:67,6,0,0
 BRDA:67,6,1,5
 BRDA:90,7,0,0
 BRDA:90,7,1,0
-BRDA:104,8,0,15
-BRDA:104,8,1,6
+BRDA:104,8,0,19
+BRDA:104,8,1,7
 BRDA:115,9,0,1
-BRDA:115,9,1,4
+BRDA:115,9,1,5
 BRDA:129,10,0,0
-BRDA:129,10,1,4
-BRDA:147,11,0,72
+BRDA:129,10,1,5
+BRDA:147,11,0,71
 BRDA:147,11,1,0
-BRDA:147,12,0,72
-BRDA:147,12,1,72
+BRDA:147,12,0,71
+BRDA:147,12,1,71
 BRDA:159,13,0,0
-BRDA:159,13,1,72
+BRDA:159,13,1,71
 BRDA:181,14,0,1
 BRDA:181,14,1,6
 BRDA:190,15,0,5
@@ -5244,7 +5241,7 @@ DA:13,24
 DA:33,24
 DA:34,24
 DA:37,24
-DA:55,24
+DA:54,24
 LF:12
 LH:12
 BRDA:33,0,0,24
@@ -5607,8 +5604,8 @@ FNDA:19,queueRelevantRowAutomations
 FNDA:19,(anonymous_1)
 FNDA:4,(anonymous_2)
 FNDA:4,(anonymous_3)
-FNDA:10,(anonymous_4)
-FNDA:3,(anonymous_5)
+FNDA:9,(anonymous_4)
+FNDA:4,(anonymous_5)
 FNDA:6,(anonymous_6)
 FNDA:6,(anonymous_7)
 DA:1,39
@@ -5640,9 +5637,9 @@ DA:49,0
 DA:51,4
 DA:55,4
 DA:61,39
-DA:66,10
+DA:66,9
 DA:69,39
-DA:74,3
+DA:74,4
 DA:77,39
 DA:82,6
 DA:85,39
@@ -6182,14 +6179,15 @@ BRDA:88,0,1,0
 BRDA:88,1,0,1
 BRDA:88,1,1,1
 BRDA:88,1,2,1
+BRDA:88,1,3,1
 BRDA:91,2,0,0
 BRDA:91,2,1,1
 BRDA:101,3,0,1
 BRDA:101,3,1,0
 BRDA:108,4,0,1
 BRDA:108,4,1,0
-BRF:11
-BRH:7
+BRF:12
+BRH:8
 end_of_record
 TN:
 SF:src/automations/steps/sendSmtpEmail.js
@@ -6584,10 +6582,10 @@ SF:src/db/newid.js
 FN:3,(anonymous_0)
 FNF:1
 FNH:1
-FNDA:946,(anonymous_0)
+FNDA:945,(anonymous_0)
 DA:1,55
 DA:3,55
-DA:4,946
+DA:4,945
 LF:3
 LH:3
 BRF:0
@@ -6635,7 +6633,7 @@ FNDA:222,(anonymous_2)
 FNDA:106,(anonymous_3)
 FNDA:16,(anonymous_4)
 FNDA:0,(anonymous_5)
-FNDA:266,(anonymous_6)
+FNDA:265,(anonymous_6)
 FNDA:3,(anonymous_7)
 FNDA:193,(anonymous_8)
 FNDA:2,(anonymous_9)
@@ -6706,8 +6704,8 @@ DA:154,55
 DA:155,0
 DA:158,0
 DA:167,55
-DA:168,266
-DA:169,266
+DA:168,265
+DA:169,265
 DA:175,55
 DA:176,3
 DA:184,55
@@ -6789,9 +6787,9 @@ BRDA:140,8,0,4
 BRDA:140,8,1,12
 BRDA:144,9,0,12
 BRDA:144,9,1,0
-BRDA:167,10,0,73
-BRDA:168,11,0,266
-BRDA:168,11,1,73
+BRDA:167,10,0,72
+BRDA:168,11,0,265
+BRDA:168,11,1,72
 BRDA:175,12,0,0
 BRDA:175,13,0,0
 BRDA:193,14,0,1
@@ -7772,15 +7770,16 @@ DA:85,0
 DA:86,0
 DA:87,0
 DA:88,0
-DA:91,190
+DA:89,0
 DA:92,190
 DA:93,190
 DA:94,190
 DA:95,190
-DA:104,190
-DA:110,7
-DA:113,190
-LF:55
+DA:96,190
+DA:105,190
+DA:111,7
+DA:114,190
+LF:56
 LH:46
 BRDA:28,0,0,1
 BRDA:28,0,1,192
@@ -7815,16 +7814,16 @@ BRDA:78,13,0,190
 BRDA:78,13,1,0
 BRDA:78,13,2,0
 BRDA:78,13,3,0
-BRDA:92,14,0,190
-BRDA:92,14,1,0
-BRDA:94,15,0,185
-BRDA:94,15,1,5
-BRDA:104,16,0,7
-BRDA:104,16,1,183
-BRDA:105,17,0,190
-BRDA:105,17,1,190
-BRDA:105,17,2,185
-BRDA:105,17,3,7
+BRDA:93,14,0,190
+BRDA:93,14,1,0
+BRDA:95,15,0,185
+BRDA:95,15,1,5
+BRDA:105,16,0,7
+BRDA:105,16,1,183
+BRDA:106,17,0,190
+BRDA:106,17,1,190
+BRDA:106,17,2,185
+BRDA:106,17,3,7
 BRF:43
 BRH:33
 end_of_record
@@ -9486,7 +9485,7 @@ FNDA:182,(anonymous_11)
 FNDA:78,(anonymous_12)
 FNDA:107,(anonymous_13)
 FNDA:2,(anonymous_14)
-FNDA:13,(anonymous_15)
+FNDA:14,(anonymous_15)
 FNDA:0,addFiles
 FNDA:0,(anonymous_17)
 FNDA:0,(anonymous_18)
@@ -9562,17 +9561,17 @@ DA:221,78
 DA:223,78
 DA:224,78
 DA:225,78
-DA:226,352
-DA:228,352
-DA:229,171
+DA:226,353
+DA:228,353
+DA:229,172
 DA:230,3
-DA:232,171
+DA:232,172
 DA:235,181
 DA:236,0
 DA:240,181
 DA:244,78
-DA:245,73
-DA:246,73
+DA:245,72
+DA:246,72
 DA:250,78
 DA:262,40
 DA:263,107
@@ -9592,12 +9591,12 @@ DA:287,107
 DA:288,27
 DA:290,107
 DA:303,40
-DA:304,13
-DA:305,13
-DA:306,13
-DA:308,13
-DA:309,13
-DA:310,13
+DA:304,14
+DA:305,14
+DA:306,14
+DA:308,14
+DA:309,14
+DA:310,14
 DA:313,0
 DA:315,0
 DA:316,0
@@ -9670,14 +9669,14 @@ BRDA:199,22,0,22
 BRDA:199,22,1,144
 BRDA:216,23,0,0
 BRDA:219,24,0,77
-BRDA:228,25,0,171
+BRDA:228,25,0,172
 BRDA:228,25,1,181
 BRDA:229,26,0,3
-BRDA:229,26,1,168
+BRDA:229,26,1,169
 BRDA:235,27,0,0
 BRDA:235,27,1,181
-BRDA:244,28,0,73
-BRDA:244,28,1,5
+BRDA:244,28,0,72
+BRDA:244,28,1,6
 BRDA:244,29,0,78
 BRDA:244,29,1,78
 BRDA:262,30,0,27
@@ -9693,9 +9692,9 @@ BRDA:287,35,0,27
 BRDA:287,35,1,80
 BRDA:290,36,0,14
 BRDA:290,36,1,93
-BRDA:305,37,0,13
+BRDA:305,37,0,14
 BRDA:305,37,1,0
-BRDA:309,38,0,13
+BRDA:309,38,0,14
 BRDA:309,38,1,0
 BRDA:315,39,0,0
 BRDA:315,39,1,0

package/dist/api/controllers/auth.js

@@ -16,7 +16,7 @@ const { getAppDB, getAppId } = require("@budibase/backend-core/context");
 exports.fetchSelf = (ctx) => __awaiter(void 0, void 0, void 0, function* () {
     let userId = ctx.user.userId || ctx.user._id;
     /* istanbul ignore next */
-    if (!userId) {
+    if (!userId || !ctx.isAuthenticated) {
         ctx.body = {};
         return;
     }

package/dist/api/controllers/public/rows.js

@@ -71,7 +71,7 @@ function read(ctx, next) {
 exports.read = read;
 function update(ctx, next) {
     return __awaiter(this, void 0, void 0, function* () {
-        ctx.request.body = yield utils_1.addRev(fixRow(ctx.request.body, ctx.params.tableId));
+        ctx.request.body = yield utils_1.addRev(fixRow(ctx.request.body, ctx.params));
         yield row_1.default.save(ctx);
         yield next();
     });
@@ -80,7 +80,7 @@ exports.update = update;
 function destroy(ctx, next) {
     return __awaiter(this, void 0, void 0, function* () {
         // set the body as expected, with the _id and _rev fields
-        ctx.request.body = yield utils_1.addRev(fixRow({ _id: ctx.params.rowId }, ctx.params.tableId));
+        ctx.request.body = yield utils_1.addRev(fixRow({ _id: ctx.params.rowId }, ctx.params));
         yield row_1.default.destroy(ctx);
         // destroy controller doesn't currently return the row as the body, need to adjust this
         // in the public API to be correct

package/dist/api/controllers/static/index.js

@@ -14,14 +14,13 @@ const { resolve, join } = require("../../../utilities/centralPath");
 const uuid = require("uuid");
 const { ObjectStoreBuckets } = require("../../../constants");
 const { processString } = require("@budibase/string-templates");
-const { getAllApps } = require("@budibase/backend-core/db");
 const { loadHandlebarsFile, NODE_MODULES_PATH, TOP_LEVEL_PATH, } = require("../../../utilities/fileSystem");
 const env = require("../../../environment");
 const { clientLibraryPath } = require("../../../utilities");
 const { upload } = require("../../../utilities/fileSystem");
 const { attachmentsRelativeURL } = require("../../../utilities");
 const { DocumentTypes } = require("../../../db/utils");
-const { getAppDB, updateAppId } = require("@budibase/backend-core/context");
+const { getAppDB, getAppId } = require("@budibase/backend-core/context");
 const AWS = require("aws-sdk");
 const AWS_REGION = env.AWS_REGION ? env.AWS_REGION : "eu-west-1";
 function prepareUpload({ s3Key, bucket, metadata, file }) {
@@ -43,18 +42,6 @@ function prepareUpload({ s3Key, bucket, metadata, file }) {
         };
     });
 }
-function getAppIdFromUrl(ctx) {
-    return __awaiter(this, void 0, void 0, function* () {
-        // the "appId" component of the URL can be the id or the custom url
-        let possibleAppUrl = `/${encodeURI(ctx.params.appId).toLowerCase()}`;
-        // search prod apps for a url that matches, exclude dev where id is always used
-        const apps = yield getAllApps({ dev: false });
-        const app = apps.filter(a => a.url && a.url.toLowerCase() === possibleAppUrl)[0];
-        const appId = app && app.appId ? app.appId : ctx.params.appId;
-        updateAppId(appId);
-        return appId;
-    });
-}
 exports.serveBuilder = function (ctx) {
     return __awaiter(this, void 0, void 0, function* () {
         let builderPath = resolve(TOP_LEVEL_PATH, "builder");
@@ -81,10 +68,10 @@ exports.uploadFile = function (ctx) {
 };
 exports.serveApp = function (ctx) {
     return __awaiter(this, void 0, void 0, function* () {
-        let appId = yield getAppIdFromUrl(ctx);
         const App = require("./templates/BudibaseApp.svelte").default;
         const db = getAppDB({ skip_setup: true });
         const appInfo = yield db.get(DocumentTypes.APP_METADATA);
+        let appId = getAppId();
         const { head, html, css } = App.render({
             title: appInfo.name,
             production: env.isProd(),

package/dist/api/controllers/table/external.js

@@ -97,8 +97,8 @@ function generateManyLinkSchema(datasource, column, table, relatedTable) {
         },
     };
     column.through = junctionTable._id;
-    column.throughFrom = primary;
-    column.throughTo = relatedPrimary;
+    column.throughFrom = relatedPrimary;
+    column.throughTo = primary;
     column.fieldName = relatedPrimary;
     return junctionTable;
 }
@@ -123,7 +123,7 @@ function generateRelatedSchema(linkColumn, table, relatedTable, columnName) {
     // is many to many
     else {
         // don't need to copy through, already got it
-        relatedSchema.fieldName = linkColumn.throughFrom;
+        relatedSchema.fieldName = linkColumn.throughTo;
         relatedSchema.throughTo = linkColumn.throughFrom;
         relatedSchema.throughFrom = linkColumn.throughTo;
     }

package/dist/api/index.js

@@ -9,7 +9,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
     });
 };
 const Router = require("@koa/router");
-const { buildAuthMiddleware, auditLog, buildTenancyMiddleware, buildAppTenancyMiddleware, } = require("@budibase/backend-core/auth");
+const { buildAuthMiddleware, auditLog, buildTenancyMiddleware, } = require("@budibase/backend-core/auth");
 const currentApp = require("../middleware/currentapp");
 const compress = require("koa-compress");
 const zlib = require("zlib");
@@ -17,6 +17,8 @@ const { mainRoutes, staticRoutes, publicRoutes } = require("./routes");
 const pkg = require("../../package.json");
 const env = require("../environment");
 const router = new Router();
+router.get("/health", ctx => (ctx.status = 200));
+router.get("/version", ctx => (ctx.body = pkg.version));
 router
     .use(compress({
     threshold: 2048,
@@ -35,8 +37,6 @@ router
     };
     yield next();
 }))
-    .use("/health", ctx => (ctx.status = 200))
-    .use("/version", ctx => (ctx.body = pkg.version))
     // re-direct before any middlewares occur
     .redirect("/", "/builder")
     .use(buildAuthMiddleware(null, {
@@ -49,8 +49,6 @@ router
     noTenancyRequired: true,
 }))
     .use(currentApp)
-    // this middleware will try to use the app ID to determine the tenancy
-    .use(buildAppTenancyMiddleware())
     .use(auditLog);
 // error handling middleware
 router.use((ctx, next) => __awaiter(void 0, void 0, void 0, function* () {
@@ -70,7 +68,6 @@ router.use((ctx, next) => __awaiter(void 0, void 0, void 0, function* () {
         }
     }
 }));
-router.get("/health", ctx => (ctx.status = 200));
 // authenticated routes
 for (let route of mainRoutes) {
     router.use(route.routes());

package/dist/api/routes/static.js

@@ -42,7 +42,6 @@ router
     .get("/builder/:file*", controller.serveBuilder)
     .post("/api/attachments/process", authorized(BUILDER), controller.uploadFile)
     .post("/api/attachments/:tableId/upload", paramResource("tableId"), authorized(PermissionTypes.TABLE, PermissionLevels.WRITE), controller.uploadFile)
-    // TODO: this likely needs to be secured in some way
-    .get("/:appId/:path*", controller.serveApp)
+    .get("/app/:appUrl/:path*", controller.serveApp)
     .post("/api/attachments/:datasourceId/url", authorized(PermissionTypes.TABLE, PermissionLevels.READ), controller.getSignedUploadURL);
 module.exports = router;

package/dist/automations/steps/queryRows.js

@@ -93,7 +93,7 @@ exports.run = function ({ inputs, appId }) {
         const { tableId, filters, sortColumn, sortOrder, limit } = inputs;
         const table = yield getTable(appId, tableId);
         let sortType = FieldTypes.STRING;
-        if (table && table.schema && sortColumn) {
+        if (table && table.schema && table.schema[sortColumn] && sortColumn) {
             const fieldType = table.schema[sortColumn].type;
             sortType =
                 fieldType === FieldTypes.NUMBER ? FieldTypes.NUMBER : FieldTypes.STRING;