@budibase/server 1.0.219-alpha.1 → 1.0.219-alpha.4

package/coverage/lcov.info

@@ -7091,19 +7091,19 @@ BRH:20
 end_of_record
 TN:
 SF:src/threads/query.js
-FN:20,(anonymous_0)
-FN:40,(anonymous_1)
-FN:50,(anonymous_2)
-FN:142,(anonymous_3)
-FN:143,(anonymous_4)
-FN:156,(anonymous_5)
-FN:172,(anonymous_6)
-FN:197,(anonymous_7)
-FN:215,(anonymous_8)
-FN:230,(anonymous_9)
-FN:239,(anonymous_10)
-FN:261,(anonymous_11)
-FN:262,(anonymous_12)
+FN:21,(anonymous_0)
+FN:41,(anonymous_1)
+FN:51,(anonymous_2)
+FN:137,(anonymous_3)
+FN:138,(anonymous_4)
+FN:151,(anonymous_5)
+FN:167,(anonymous_6)
+FN:198,(anonymous_7)
+FN:216,(anonymous_8)
+FN:231,(anonymous_9)
+FN:240,(anonymous_10)
+FN:262,(anonymous_11)
+FN:263,(anonymous_12)
 FNF:13
 FNH:10
 FNDA:22,(anonymous_0)
@@ -7127,9 +7127,9 @@ DA:5,2
 DA:6,2
 DA:10,2
 DA:11,2
-DA:13,2
-DA:17,2
-DA:21,22
+DA:12,2
+DA:14,2
+DA:18,2
 DA:22,22
 DA:23,22
 DA:24,22
@@ -7138,177 +7138,173 @@ DA:26,22
 DA:27,22
 DA:28,22
 DA:29,22
-DA:31,22
-DA:35,22
+DA:30,22
+DA:32,22
 DA:36,22
 DA:37,22
-DA:41,23
-DA:43,23
+DA:38,22
+DA:42,23
 DA:44,23
-DA:45,2
-DA:48,21
-DA:49,0
-DA:51,0
-DA:56,21
-DA:59,21
-DA:63,21
+DA:45,23
+DA:46,2
+DA:49,21
+DA:50,0
+DA:52,0
+DA:57,21
+DA:60,21
 DA:64,21
-DA:67,21
-DA:68,1
-DA:76,21
-DA:77,2
-DA:79,19
-DA:83,21
-DA:84,0
-DA:87,21
+DA:65,21
+DA:68,21
+DA:69,1
+DA:77,21
+DA:78,2
+DA:80,19
+DA:84,21
+DA:85,0
 DA:88,21
 DA:89,21
 DA:90,21
 DA:91,21
 DA:92,21
-DA:93,19
+DA:93,21
 DA:94,19
 DA:95,19
 DA:96,19
-DA:100,21
-DA:101,0
-DA:105,0
-DA:109,21
-DA:110,1
+DA:97,19
+DA:101,21
+DA:102,0
+DA:106,0
+DA:110,21
+DA:111,1
 DA:116,0
-DA:117,0
-DA:119,0
-DA:122,0
-DA:126,1
-DA:127,1
-DA:128,1
+DA:118,0
+DA:121,1
+DA:122,1
+DA:123,1
+DA:127,20
+DA:128,0
 DA:132,20
-DA:133,0
+DA:133,18
 DA:137,20
-DA:138,18
+DA:138,0
 DA:142,20
-DA:143,0
-DA:147,20
-DA:149,20
-DA:150,0
-DA:153,20
-DA:157,5
-DA:158,5
-DA:159,5
-DA:160,5
-DA:173,0
-DA:174,0
-DA:176,0
-DA:177,0
-DA:178,0
-DA:181,0
-DA:189,0
+DA:144,20
+DA:145,0
+DA:148,20
+DA:152,5
+DA:153,5
+DA:154,5
+DA:155,5
+DA:168,0
+DA:169,0
+DA:171,0
+DA:172,0
+DA:175,0
+DA:183,0
+DA:184,0
+DA:185,0
+DA:186,0
 DA:190,0
-DA:191,0
-DA:194,0
-DA:198,6
+DA:195,0
 DA:199,6
 DA:200,6
 DA:201,6
 DA:202,6
-DA:203,5
-DA:207,5
+DA:203,6
+DA:204,5
 DA:208,5
-DA:210,1
-DA:212,6
-DA:216,21
+DA:209,5
+DA:211,1
+DA:213,6
 DA:217,21
-DA:218,0
-DA:220,21
+DA:218,21
+DA:219,0
 DA:221,21
 DA:222,21
-DA:223,2
+DA:223,21
 DA:224,2
-DA:227,21
-DA:229,16
+DA:225,2
+DA:228,21
 DA:230,16
-DA:232,6
-DA:233,0
-DA:236,6
+DA:231,16
+DA:233,6
+DA:234,0
 DA:237,6
-DA:239,16
+DA:238,6
 DA:240,16
 DA:241,16
-DA:242,6
+DA:242,16
 DA:243,6
-DA:254,6
-DA:257,21
-DA:261,2
-DA:262,17
+DA:244,6
+DA:255,6
+DA:258,21
+DA:262,2
 DA:263,17
 DA:264,17
 DA:265,17
-DA:266,15
-DA:268,2
+DA:266,17
+DA:267,15
+DA:269,2
 LF:126
-LH:102
-BRDA:20,0,0,17
-BRDA:44,1,0,2
-BRDA:44,1,1,21
-BRDA:48,2,0,0
-BRDA:48,2,1,21
-BRDA:67,3,0,1
-BRDA:67,3,1,20
-BRDA:76,4,0,2
-BRDA:76,4,1,19
-BRDA:83,5,0,0
-BRDA:83,5,1,21
-BRDA:92,6,0,19
-BRDA:92,6,1,2
-BRDA:100,7,0,0
-BRDA:100,7,1,21
-BRDA:109,8,0,1
-BRDA:109,8,1,20
-BRDA:109,9,0,21
-BRDA:109,9,1,19
-BRDA:109,9,2,1
-BRDA:110,10,0,0
-BRDA:110,10,1,1
-BRDA:111,11,0,1
-BRDA:111,11,1,0
-BRDA:111,11,2,0
-BRDA:119,12,0,0
-BRDA:119,12,1,0
-BRDA:119,13,0,0
-BRDA:119,13,1,0
-BRDA:132,14,0,0
-BRDA:132,14,1,20
-BRDA:137,15,0,18
-BRDA:137,15,1,2
-BRDA:142,16,0,0
-BRDA:142,16,1,20
-BRDA:149,17,0,0
-BRDA:149,17,1,20
-BRDA:176,18,0,0
-BRDA:176,18,1,0
-BRDA:176,19,0,0
-BRDA:176,19,1,0
-BRDA:189,20,0,0
-BRDA:189,20,1,0
-BRDA:202,21,0,5
-BRDA:202,21,1,1
-BRDA:203,22,0,0
-BRDA:203,22,1,5
-BRDA:217,23,0,0
-BRDA:217,23,1,21
-BRDA:217,24,0,21
-BRDA:217,24,1,21
-BRDA:220,25,0,21
-BRDA:220,25,1,20
-BRDA:221,26,0,21
-BRDA:221,26,1,10
-BRDA:223,27,0,2
-BRDA:223,27,1,0
-BRDA:227,28,0,16
-BRDA:227,28,1,5
-BRDA:232,29,0,0
-BRDA:232,29,1,6
-BRF:61
+LH:103
+BRDA:21,0,0,17
+BRDA:45,1,0,2
+BRDA:45,1,1,21
+BRDA:49,2,0,0
+BRDA:49,2,1,21
+BRDA:68,3,0,1
+BRDA:68,3,1,20
+BRDA:77,4,0,2
+BRDA:77,4,1,19
+BRDA:84,5,0,0
+BRDA:84,5,1,21
+BRDA:93,6,0,19
+BRDA:93,6,1,2
+BRDA:101,7,0,0
+BRDA:101,7,1,21
+BRDA:110,8,0,1
+BRDA:110,8,1,20
+BRDA:110,9,0,21
+BRDA:110,9,1,19
+BRDA:110,9,2,1
+BRDA:111,10,0,0
+BRDA:111,10,1,1
+BRDA:112,11,0,1
+BRDA:112,11,1,0
+BRDA:112,11,2,0
+BRDA:127,12,0,0
+BRDA:127,12,1,20
+BRDA:132,13,0,18
+BRDA:132,13,1,2
+BRDA:137,14,0,0
+BRDA:137,14,1,20
+BRDA:144,15,0,0
+BRDA:144,15,1,20
+BRDA:171,16,0,0
+BRDA:171,16,1,0
+BRDA:171,17,0,0
+BRDA:171,17,1,0
+BRDA:183,18,0,0
+BRDA:183,18,1,0
+BRDA:203,19,0,5
+BRDA:203,19,1,1
+BRDA:204,20,0,0
+BRDA:204,20,1,5
+BRDA:218,21,0,0
+BRDA:218,21,1,21
+BRDA:218,22,0,21
+BRDA:218,22,1,21
+BRDA:221,23,0,21
+BRDA:221,23,1,20
+BRDA:222,24,0,21
+BRDA:222,24,1,10
+BRDA:224,25,0,2
+BRDA:224,25,1,0
+BRDA:228,26,0,16
+BRDA:228,26,1,5
+BRDA:233,27,0,0
+BRDA:233,27,1,6
+BRF:57
 BRH:38
 end_of_record
 TN:

package/dist/package.json

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

package/dist/threads/query.js

@@ -15,6 +15,7 @@ const { integrations } = require("../integrations");
 const { processStringSync } = require("@budibase/string-templates");
 const { doInAppContext, getAppDB } = require("@budibase/backend-core/context");
 const { refreshOAuthToken, updateUserOAuth, } = require("@budibase/backend-core/auth");
+const { user: userCache } = require("@budibase/backend-core/cache");
 const { getGlobalIDFromUserMetadataID } = require("../db/utils");
 const { isSQL } = require("../integrations/utils");
 const { enrichQueryFields, interpolateSQL, } = require("../integrations/queries/sql");
@@ -95,14 +96,9 @@ class QueryRunner {
                 if (((_a = this.ctx.user) === null || _a === void 0 ? void 0 : _a.provider) &&
                     info.code === 401 &&
                     !this.hasRefreshedOAuth) {
+                    yield this.refreshOAuth2(this.ctx);
                     // Attempt to refresh the access token from the provider
                     this.hasRefreshedOAuth = true;
-                    const authResponse = yield this.refreshOAuth2(this.ctx);
-                    if (!authResponse || authResponse.err) {
-                        // In this event the user may have oAuth issues that
-                        // could require re-authenticating with their provider.
-                        throw new Error("OAuth2 access token could not be refreshed");
-                    }
                 }
                 this.hasRerun = true;
                 yield threadUtils.invalidateDynamicVariables(this.cachedVariables);
@@ -147,15 +143,20 @@ class QueryRunner {
             const { oauth2, providerType, _id } = ctx.user;
             const { configId } = ctx.auth;
             if (!providerType || !(oauth2 === null || oauth2 === void 0 ? void 0 : oauth2.refreshToken)) {
-                console.error("No refresh token found for authenticated user");
-                return;
+                throw new Error("No refresh token found for authenticated user");
             }
             const resp = yield refreshOAuthToken(oauth2.refreshToken, providerType, configId);
             // Refresh session flow. Should be in same location as refreshOAuthToken
             // There are several other properties available in 'resp'
-            if (!resp.error) {
+            if (!resp.err) {
                 const globalUserId = getGlobalIDFromUserMetadataID(_id);
                 yield updateUserOAuth(globalUserId, resp);
+                this.ctx.user = yield userCache.getUser(globalUserId);
+            }
+            else {
+                // In this event the user may have oAuth issues that
+                // could require re-authenticating with their provider.
+                throw new Error("OAuth2 access token could not be refreshed: " + resp.err.toString());
             }
             return resp;
         });