@budibase/worker 1.3.5 → 1.3.8

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@budibase/worker",
   "email": "hi@budibase.com",
-  "version": "1.3.5",
+  "version": "1.3.8",
   "description": "Budibase background service",
   "main": "src/index.ts",
   "repository": {
@@ -35,10 +35,10 @@
   "author": "Budibase",
   "license": "GPL-3.0",
   "dependencies": {
-    "@budibase/backend-core": "^1.3.5",
-    "@budibase/pro": "1.3.4",
-    "@budibase/string-templates": "^1.3.5",
-    "@budibase/types": "^1.3.5",
+    "@budibase/backend-core": "^1.3.8",
+    "@budibase/pro": "1.3.7",
+    "@budibase/string-templates": "^1.3.8",
+    "@budibase/types": "^1.3.8",
     "@koa/router": "8.0.8",
     "@sentry/node": "6.17.7",
     "@techpass/passport-openidconnect": "0.3.2",
@@ -103,5 +103,5 @@
       "./scripts/jestSetup.js"
     ]
   },
-  "gitHead": "e3ac6893b3d1b008fb89373c1b8759475f22aabd"
+  "gitHead": "1142dff43086090a96ade22cdad43d0a8ace988a"
 }

package/src/sdk/users/users.ts

@@ -189,23 +189,34 @@ export const save = async (
   const tenantId = tenancy.getTenantId()
   const db = tenancy.getGlobalDB()
   let { email, _id } = user
+  if (!email && !_id) {
+    throw new Error("_id or email is required")
+  }
 
   let dbUser: User | undefined
   if (_id) {
     // try to get existing user from db
-    dbUser = (await db.get(_id)) as User
-    if (email && dbUser.email !== email) {
-      throw "Email address cannot be changed"
+    try {
+      dbUser = (await db.get(_id)) as User
+      if (email && dbUser.email !== email) {
+        throw "Email address cannot be changed"
+      }
+      email = dbUser.email
+    } catch (e: any) {
+      if (e.status === 404) {
+        // do nothing, save this new user with the id specified - required for SSO auth
+      } else {
+        throw e
+      }
     }
-    email = dbUser.email
-  } else if (email) {
+  }
+
+  if (!dbUser && email) {
     // no id was specified - load from email instead
     dbUser = await usersCore.getGlobalUserByEmail(email)
     if (dbUser && dbUser._id !== _id) {
       throw `Unavailable`
     }
-  } else {
-    throw new Error("_id or email is required")
   }
 
   await validateUniqueUser(email, tenantId)