@budibase/server 1.0.105 → 1.0.108

package/coverage/lcov.info

@@ -5847,31 +5847,34 @@ BRH:31
 end_of_record
 TN:
 SF:src/automations/steps/bash.js
-FN:41,(anonymous_0)
+FN:42,(anonymous_0)
 FNF:1
 FNH:0
 FNDA:0,(anonymous_0)
 DA:1,40
 DA:2,40
 DA:3,40
-DA:5,40
-DA:41,40
-DA:42,0
+DA:4,40
+DA:6,40
+DA:42,40
 DA:43,0
-DA:48,0
+DA:44,0
 DA:49,0
-DA:52,0
+DA:50,0
 DA:53,0
 DA:54,0
-DA:56,0
-DA:57,0
+DA:55,0
+DA:59,0
 DA:60,0
-DA:65,0
-LF:16
-LH:5
-BRDA:42,0,0,0
-BRDA:42,0,1,0
-BRF:2
+DA:63,0
+DA:68,0
+LF:17
+LH:6
+BRDA:43,0,0,0
+BRDA:43,0,1,0
+BRDA:56,1,0,0
+BRDA:56,1,1,0
+BRF:4
 BRH:0
 end_of_record
 TN:
@@ -8374,40 +8377,42 @@ BRH:10
 end_of_record
 TN:
 SF:src/threads/query.js
-FN:13,(anonymous_0)
-FN:30,(anonymous_1)
-FN:69,(anonymous_2)
-FN:93,(anonymous_3)
-FN:161,(anonymous_4)
-FN:162,(anonymous_5)
-FN:175,(anonymous_6)
-FN:191,(anonymous_7)
-FN:209,(anonymous_8)
-FN:224,(anonymous_9)
+FN:15,(anonymous_0)
+FN:32,(anonymous_1)
+FN:49,(anonymous_2)
+FN:74,(anonymous_3)
+FN:102,(anonymous_4)
+FN:170,(anonymous_5)
+FN:171,(anonymous_6)
+FN:184,(anonymous_7)
+FN:200,(anonymous_8)
+FN:218,(anonymous_9)
 FN:233,(anonymous_10)
-FN:254,(anonymous_11)
-FN:296,(anonymous_12)
-FN:297,(anonymous_13)
-FN:300,(anonymous_14)
-FN:303,(anonymous_15)
-FNF:16
+FN:242,(anonymous_11)
+FN:263,(anonymous_12)
+FN:305,(anonymous_13)
+FN:306,(anonymous_14)
+FN:309,(anonymous_15)
+FN:312,(anonymous_16)
+FNF:17
 FNH:14
 FNDA:15,(anonymous_0)
 FNDA:2,(anonymous_1)
 FNDA:0,(anonymous_2)
-FNDA:16,(anonymous_3)
-FNDA:13,(anonymous_4)
-FNDA:0,(anonymous_5)
-FNDA:5,(anonymous_6)
-FNDA:6,(anonymous_7)
-FNDA:14,(anonymous_8)
-FNDA:6,(anonymous_9)
+FNDA:0,(anonymous_3)
+FNDA:16,(anonymous_4)
+FNDA:13,(anonymous_5)
+FNDA:0,(anonymous_6)
+FNDA:5,(anonymous_7)
+FNDA:6,(anonymous_8)
+FNDA:14,(anonymous_9)
 FNDA:6,(anonymous_10)
-FNDA:12,(anonymous_11)
-FNDA:10,(anonymous_12)
+FNDA:6,(anonymous_11)
+FNDA:12,(anonymous_12)
 FNDA:10,(anonymous_13)
-FNDA:8,(anonymous_14)
-FNDA:2,(anonymous_15)
+FNDA:10,(anonymous_14)
+FNDA:8,(anonymous_15)
+FNDA:2,(anonymous_16)
 DA:1,2
 DA:2,2
 DA:3,2
@@ -8415,8 +8420,7 @@ DA:4,2
 DA:8,2
 DA:9,2
 DA:10,2
-DA:14,15
-DA:15,15
+DA:12,2
 DA:16,15
 DA:17,15
 DA:18,15
@@ -8424,205 +8428,214 @@ DA:19,15
 DA:20,15
 DA:21,15
 DA:22,15
-DA:26,15
-DA:27,15
-DA:31,2
-DA:32,2
+DA:23,15
+DA:24,15
+DA:28,15
+DA:29,15
 DA:33,2
-DA:35,0
-DA:36,0
+DA:34,2
+DA:35,2
 DA:37,0
 DA:38,0
+DA:39,0
 DA:40,0
-DA:41,0
-DA:43,0
-DA:44,0
-DA:45,0
-DA:46,0
+DA:42,0
 DA:47,0
-DA:48,0
 DA:49,0
-DA:59,0
-DA:60,0
-DA:62,0
+DA:50,0
+DA:52,0
+DA:53,0
+DA:54,0
+DA:55,0
+DA:56,0
 DA:66,0
+DA:67,0
 DA:69,0
 DA:73,0
-DA:75,0
+DA:74,0
+DA:77,0
 DA:78,0
-DA:79,0
-DA:81,0
+DA:80,0
 DA:82,0
-DA:83,0
 DA:84,0
-DA:86,0
-DA:89,0
+DA:87,0
+DA:88,0
 DA:90,0
-DA:94,16
-DA:95,16
-DA:96,16
-DA:97,2
-DA:99,14
-DA:102,14
-DA:105,14
+DA:91,0
+DA:92,0
+DA:93,0
+DA:95,0
+DA:98,0
+DA:99,0
+DA:103,16
+DA:104,16
+DA:105,16
 DA:106,2
-DA:108,12
-DA:112,14
-DA:113,0
-DA:116,14
-DA:117,14
-DA:118,14
-DA:119,14
-DA:120,14
+DA:108,14
+DA:111,14
+DA:114,14
+DA:115,2
+DA:117,12
 DA:121,14
-DA:122,12
-DA:123,12
-DA:124,12
-DA:125,12
+DA:122,0
+DA:125,14
+DA:126,14
+DA:127,14
+DA:128,14
 DA:129,14
-DA:130,0
-DA:134,0
+DA:130,14
+DA:131,12
+DA:132,12
+DA:133,12
+DA:134,12
 DA:138,14
-DA:144,1
-DA:146,1
-DA:147,1
-DA:151,13
-DA:152,0
-DA:156,13
-DA:157,11
-DA:161,13
-DA:162,0
-DA:166,13
-DA:168,13
-DA:169,0
-DA:172,13
-DA:176,5
-DA:177,5
-DA:178,5
-DA:179,5
-DA:192,6
-DA:193,6
-DA:194,6
-DA:195,6
-DA:196,6
-DA:197,5
-DA:201,5
-DA:202,5
-DA:204,1
-DA:206,6
-DA:210,14
-DA:211,14
-DA:212,0
-DA:214,14
-DA:215,14
-DA:216,14
-DA:217,2
-DA:218,2
-DA:221,14
-DA:223,9
-DA:224,9
-DA:226,6
-DA:227,0
-DA:230,6
-DA:231,6
+DA:139,0
+DA:143,0
+DA:147,14
+DA:153,1
+DA:155,1
+DA:156,1
+DA:160,13
+DA:161,0
+DA:165,13
+DA:166,11
+DA:170,13
+DA:171,0
+DA:175,13
+DA:177,13
+DA:178,0
+DA:181,13
+DA:185,5
+DA:186,5
+DA:187,5
+DA:188,5
+DA:201,6
+DA:202,6
+DA:203,6
+DA:204,6
+DA:205,6
+DA:206,5
+DA:210,5
+DA:211,5
+DA:213,1
+DA:215,6
+DA:219,14
+DA:220,14
+DA:221,0
+DA:223,14
+DA:224,14
+DA:225,14
+DA:226,2
+DA:227,2
+DA:230,14
+DA:232,9
 DA:233,9
-DA:234,9
-DA:235,9
-DA:236,6
-DA:237,6
-DA:248,6
-DA:251,14
-DA:255,12
-DA:258,12
-DA:259,19
-DA:260,0
-DA:262,19
-DA:264,0
-DA:265,19
-DA:267,19
+DA:235,6
+DA:236,0
+DA:239,6
+DA:240,6
+DA:242,9
+DA:243,9
+DA:244,9
+DA:245,6
+DA:246,6
+DA:257,6
+DA:260,14
+DA:264,12
+DA:267,12
+DA:268,19
+DA:269,0
+DA:271,19
 DA:273,0
-DA:276,12
-DA:281,0
+DA:274,19
+DA:276,19
 DA:282,0
+DA:285,12
 DA:290,0
-DA:292,12
-DA:296,2
-DA:297,10
-DA:298,10
-DA:299,10
-DA:301,8
-DA:304,2
-LF:144
-LH:101
-BRDA:13,0,0,10
-BRDA:32,1,0,2
-BRDA:32,1,1,0
-BRDA:45,2,0,0
-BRDA:45,2,1,0
-BRDA:59,3,0,0
-BRDA:59,3,1,0
-BRDA:83,4,0,0
-BRDA:83,4,1,0
-BRDA:96,5,0,2
-BRDA:96,5,1,14
-BRDA:105,6,0,2
-BRDA:105,6,1,12
-BRDA:112,7,0,0
-BRDA:112,7,1,14
-BRDA:121,8,0,12
-BRDA:121,8,1,2
-BRDA:129,9,0,0
-BRDA:129,9,1,14
-BRDA:138,10,0,1
-BRDA:138,10,1,13
-BRDA:139,11,0,14
-BRDA:139,11,1,12
-BRDA:139,11,2,1
-BRDA:139,11,3,1
-BRDA:151,12,0,0
-BRDA:151,12,1,13
-BRDA:156,13,0,11
-BRDA:156,13,1,2
-BRDA:161,14,0,0
-BRDA:161,14,1,13
-BRDA:168,15,0,0
-BRDA:168,15,1,13
-BRDA:196,16,0,5
-BRDA:196,16,1,1
-BRDA:197,17,0,0
-BRDA:197,17,1,5
-BRDA:211,18,0,0
-BRDA:211,18,1,14
-BRDA:211,19,0,14
-BRDA:211,19,1,14
-BRDA:214,20,0,14
-BRDA:214,20,1,13
-BRDA:215,21,0,14
-BRDA:215,21,1,3
-BRDA:217,22,0,2
-BRDA:217,22,1,0
-BRDA:221,23,0,9
-BRDA:221,23,1,5
-BRDA:226,24,0,0
-BRDA:226,24,1,6
-BRDA:254,25,0,0
-BRDA:255,26,0,0
-BRDA:255,26,1,12
-BRDA:259,27,0,0
-BRDA:259,27,1,19
-BRDA:262,28,0,0
-BRDA:262,28,1,19
-BRDA:265,29,0,19
-BRDA:265,29,1,0
-BRDA:276,30,0,0
-BRDA:276,30,1,12
-BRDA:277,31,0,12
-BRDA:277,31,1,12
-BRDA:277,31,2,12
-BRDA:283,32,0,0
-BRDA:283,32,1,0
-BRDA:283,32,2,0
-BRF:68
+DA:291,0
+DA:299,0
+DA:301,12
+DA:305,2
+DA:306,10
+DA:307,10
+DA:308,10
+DA:310,8
+DA:313,2
+LF:148
+LH:102
+BRDA:15,0,0,10
+BRDA:34,1,0,2
+BRDA:34,1,1,0
+BRDA:47,2,0,0
+BRDA:47,2,1,0
+BRDA:52,3,0,0
+BRDA:52,3,1,0
+BRDA:66,4,0,0
+BRDA:66,4,1,0
+BRDA:77,5,0,0
+BRDA:77,5,1,0
+BRDA:92,6,0,0
+BRDA:92,6,1,0
+BRDA:105,7,0,2
+BRDA:105,7,1,14
+BRDA:114,8,0,2
+BRDA:114,8,1,12
+BRDA:121,9,0,0
+BRDA:121,9,1,14
+BRDA:130,10,0,12
+BRDA:130,10,1,2
+BRDA:138,11,0,0
+BRDA:138,11,1,14
+BRDA:147,12,0,1
+BRDA:147,12,1,13
+BRDA:148,13,0,14
+BRDA:148,13,1,12
+BRDA:148,13,2,1
+BRDA:148,13,3,1
+BRDA:160,14,0,0
+BRDA:160,14,1,13
+BRDA:165,15,0,11
+BRDA:165,15,1,2
+BRDA:170,16,0,0
+BRDA:170,16,1,13
+BRDA:177,17,0,0
+BRDA:177,17,1,13
+BRDA:205,18,0,5
+BRDA:205,18,1,1
+BRDA:206,19,0,0
+BRDA:206,19,1,5
+BRDA:220,20,0,0
+BRDA:220,20,1,14
+BRDA:220,21,0,14
+BRDA:220,21,1,14
+BRDA:223,22,0,14
+BRDA:223,22,1,13
+BRDA:224,23,0,14
+BRDA:224,23,1,3
+BRDA:226,24,0,2
+BRDA:226,24,1,0
+BRDA:230,25,0,9
+BRDA:230,25,1,5
+BRDA:235,26,0,0
+BRDA:235,26,1,6
+BRDA:263,27,0,0
+BRDA:264,28,0,0
+BRDA:264,28,1,12
+BRDA:268,29,0,0
+BRDA:268,29,1,19
+BRDA:271,30,0,0
+BRDA:271,30,1,19
+BRDA:274,31,0,19
+BRDA:274,31,1,0
+BRDA:285,32,0,0
+BRDA:285,32,1,12
+BRDA:286,33,0,12
+BRDA:286,33,1,12
+BRDA:286,33,2,12
+BRDA:292,34,0,0
+BRDA:292,34,1,0
+BRDA:292,34,2,0
+BRF:72
 BRH:43
 end_of_record
 TN:

package/dist/automations/steps/bash.js

@@ -11,6 +11,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 const { execSync } = require("child_process");
 const { processStringSync } = require("@budibase/string-templates");
 const automationUtils = require("../automationUtils");
+const environment = require("../../environment");
 exports.definition = {
     name: "Bash Scripting",
     tagline: "Execute a bash command",
@@ -57,7 +58,9 @@ exports.run = function ({ inputs, context }) {
             const command = processStringSync(inputs.code, context);
             let stdout, success = true;
             try {
-                stdout = execSync(command, { timeout: 500 }).toString();
+                stdout = execSync(command, {
+                    timeout: environment.QUERY_THREAD_TIMEOUT || 500,
+                }).toString();
             }
             catch (err) {
                 stdout = err.message;

package/dist/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@budibase/server",
     "email": "hi@budibase.com",
-    "version": "1.0.105-alpha.0",
+    "version": "1.0.107",
     "description": "Budibase Web Server",
     "main": "src/index.ts",
     "repository": {
@@ -68,9 +68,9 @@
     "license": "GPL-3.0",
     "dependencies": {
         "@apidevtools/swagger-parser": "^10.0.3",
-        "@budibase/backend-core": "^1.0.105-alpha.0",
-        "@budibase/client": "^1.0.105-alpha.0",
-        "@budibase/string-templates": "^1.0.105-alpha.0",
+        "@budibase/backend-core": "^1.0.107",
+        "@budibase/client": "^1.0.107",
+        "@budibase/string-templates": "^1.0.107",
         "@bull-board/api": "^3.7.0",
         "@bull-board/koa": "^3.7.0",
         "@elastic/elasticsearch": "7.10.0",

package/dist/threads/query.js

@@ -15,6 +15,7 @@ const { integrations } = require("../integrations");
 const { processStringSync, findHBSBlocks, } = require("@budibase/string-templates");
 const { doInAppContext, getAppDB } = require("@budibase/backend-core/context");
 const { isSQL } = require("../integrations/utils");
+const CONST_CHAR_REGEX = new RegExp("'[^']*'", "g");
 class QueryRunner {
     constructor(input, flags = { noRecursiveQuery: false }) {
         this.datasource = input.datasource;
@@ -41,16 +42,17 @@ class QueryRunner {
         let variables = [], arrays = [];
         for (let binding of bindings) {
             // look for array/list operations in the SQL statement, which will need handled later
-            const listRegex = new RegExp(`(in|IN|In|iN)( )+${binding}`);
-            const listRegexMatch = sql.match(listRegex);
+            const listRegexMatch = sql.match(new RegExp(`(in|IN|In|iN)( )+[(]?${binding}[)]?`));
             // check if the variable was used as part of a string concat e.g. 'Hello {{binding}}'
-            const charConstRegex = new RegExp(`'[^']*${binding}[^']*'`);
-            const charConstMatch = sql.match(charConstRegex);
-            if (charConstMatch) {
-                let [part1, part2] = charConstMatch[0].split(binding);
+            // start by finding all the instances of const character strings
+            const charConstMatch = sql.match(CONST_CHAR_REGEX) || [];
+            // now look within them to see if a binding is used
+            const charConstBindingMatch = charConstMatch.find(string => string.match(new RegExp(`'[^']*${binding}[^']*'`)));
+            if (charConstBindingMatch) {
+                let [part1, part2] = charConstBindingMatch.split(binding);
                 part1 = `'${part1.substring(1)}'`;
                 part2 = `'${part2.substring(0, part2.length - 1)}'`;
-                sql = sql.replace(charConstMatch[0], integration.getStringConcat([
+                sql = sql.replace(charConstBindingMatch, integration.getStringConcat([
                     part1,
                     integration.getBindingIdentifier(),
                     part2,
@@ -62,9 +64,14 @@ class QueryRunner {
                 // determine the length of the array
                 const value = this.enrichQueryFields([binding], parameters)[0].split(",");
                 // build a string like ($1, $2, $3)
-                sql = sql.replace(binding, `(${Array.apply(null, Array(value.length))
+                let replacement = `${Array.apply(null, Array(value.length))
                     .map(() => integration.getBindingIdentifier())
-                    .join(",")})`);
+                    .join(",")}`;
+                // check if parentheses are needed
+                if (!listRegexMatch[0].includes(`(${binding})`)) {
+                    replacement = `(${replacement})`;
+                }
+                sql = sql.replace(binding, replacement);
             }
             else {
                 sql = sql.replace(binding, integration.getBindingIdentifier());