@budibase/worker 2.4.5-alpha.1 → 2.4.6-alpha.0

package/package.json

@@ -1,7 +1,8 @@
 {
   "name": "@budibase/worker",
   "email": "hi@budibase.com",
-  "version": "2.4.5-alpha.1",
+  "version": "2.4.6-alpha.0",
+  "description": "Budibase background service",
   "main": "src/index.ts",
   "repository": {
     "type": "git",
@@ -35,10 +36,10 @@
   "author": "Budibase",
   "license": "GPL-3.0",
   "dependencies": {
-    "@budibase/backend-core": "2.4.5-alpha.1",
-    "@budibase/pro": "2.4.5-alpha.0",
-    "@budibase/string-templates": "2.4.5-alpha.1",
-    "@budibase/types": "2.4.5-alpha.1",
+    "@budibase/backend-core": "2.4.6-alpha.0",
+    "@budibase/pro": "2.4.5",
+    "@budibase/string-templates": "2.4.6-alpha.0",
+    "@budibase/types": "2.4.6-alpha.0",
     "@koa/router": "8.0.8",
     "@sentry/node": "6.17.7",
     "@techpass/passport-openidconnect": "0.3.2",
@@ -100,5 +101,5 @@
     "typescript": "4.7.3",
     "update-dotenv": "1.1.1"
   },
-  "gitHead": "85f8b761dd827719c26b5e8f73561d3a1ae617be"
+  "gitHead": "4f53f6ab3082d330b95a9ed27ccc04c0f79889c0"
 }

package/src/api/routes/global/tests/users.spec.ts

@@ -4,6 +4,7 @@ jest.mock("nodemailer")
 import { TestConfiguration, mocks, structures } from "../../../../tests"
 const sendMailMock = mocks.email.mock()
 import { events, tenancy, accounts as _accounts } from "@budibase/backend-core"
+import * as userSdk from "../../../../sdk/users"
 
 const accounts = jest.mocked(_accounts)
 
@@ -468,6 +469,20 @@ describe("/api/global/users", () => {
         config.authHeaders(nonAdmin)
       )
     })
+
+    describe("sso users", () => {
+      function createSSOUser() {
+        return config.doInTenant(() => {
+          const user = structures.users.ssoUser()
+          return userSdk.save(user, { requirePassword: false })
+        })
+      }
+
+      it("should be able to update an sso user that has no password", async () => {
+        const user = await createSSOUser()
+        await config.api.users.saveUser(user)
+      })
+    })
   })
 
   describe("POST /api/global/users/bulk (delete)", () => {

package/src/sdk/users/users.ts

@@ -131,6 +131,11 @@ const buildUser = async (
 ): Promise<User> => {
   let { password, _id } = user
 
+  // don't require a password if the db user doesn't already have one
+  if (dbUser && !dbUser.password) {
+    opts.requirePassword = false
+  }
+
   let hashedPassword
   if (password) {
     if (await isPreventPasswordActions(user)) {