@budibase/worker 3.18.1 → 3.18.2

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@budibase/worker",
   "email": "hi@budibase.com",
-  "version": "3.18.1",
+  "version": "3.18.2",
   "description": "Budibase background service",
   "main": "src/index.ts",
   "repository": {
@@ -109,5 +109,5 @@
       }
     }
   },
-  "gitHead": "cac7191488dabe0144f8f427686875c91cc7a548"
+  "gitHead": "43d59f5579dd910575d8aebefb40fae25570c05d"
 }

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

@@ -17,10 +17,14 @@ import nock from "nock"
 import * as jwt from "jsonwebtoken"
 
 function getAuthCookie(response: Response) {
-  return response.headers["set-cookie"]
-    .find((s: string) => s.startsWith(`${constants.Cookie.Auth}=`))
-    .split("=")[1]
-    .split(";")[0]
+  const cookies = response.headers["set-cookie"]
+  if (!cookies) throw new Error("No cookies found")
+  const cookieArray = Array.isArray(cookies) ? cookies : [cookies]
+  const authCookie = cookieArray.find((s: string) =>
+    s.startsWith(`${constants.Cookie.Auth}=`)
+  )
+  if (!authCookie) throw new Error("No auth cookie found")
+  return authCookie.split("=")[1].split(";")[0]
 }
 
 const expectSetAuthCookie = (response: Response) => {
@@ -302,9 +306,10 @@ describe("/api/global/auth", () => {
         const res = await config.api.configs.getOIDCConfig(configId)
 
         expect(res.status).toBe(302)
-        const location: string = res.get("location")
+        const location = res.get("location")
+        expect(location).toBeDefined()
         expect(
-          location.startsWith(
+          location!.startsWith(
             `http://example.com/auth?response_type=code&client_id=clientId&redirect_uri=http%3A%2F%2Flocalhost%3A10000%2Fapi%2Fglobal%2Fauth%2F${config.tenantId}%2Foidc%2Fcallback&scope=openid%20profile%20email%20offline_access`
           )
         ).toBe(true)
@@ -356,7 +361,7 @@ describe("/api/global/auth", () => {
         expect(events.auth.login).toHaveBeenCalledWith("oidc", email)
         expect(events.auth.login).toHaveBeenCalledTimes(1)
         expect(res.status).toBe(302)
-        const location: string = res.get("location")
+        const location = res.get("location")
         expect(location).toBe("/")
         expectSetAuthCookie(res)
       })