Update MultiUserClient to handle additional parameter in beginConnection

redactive-ai · Sep 20, 2024 · 343402c · 343402c
1 parent 10a8089
commit 343402c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/sdks/node/src/multiUserClient.test.ts b/sdks/node/src/multiUserClient.test.ts
@@ -42,7 +42,13 @@ describe("MultiUserClient", () => {
     const result = await multiUserClient.getBeginConnectionUrl(userId, provider);
 
     expect(result).toBe(url);
-    expect(mockAuthClient.beginConnection).toHaveBeenCalledWith(provider, "http://callback.uri", undefined, state);
+    expect(mockAuthClient.beginConnection).toHaveBeenCalledWith(
+      provider,
+      "http://callback.uri",
+      undefined,
+      undefined,
+      state
+    );
     expect(writeUserData).toHaveBeenCalledWith(userId, { signInState: state });
   });
 

diff --git a/sdks/node/src/multiUserClient.ts b/sdks/node/src/multiUserClient.ts
@@ -43,7 +43,7 @@ export class MultiUserClient {
 
   async getBeginConnectionUrl(userId: string, provider: string): Promise<string> {
     const state = randomUUID();
-    const url = await this.authClient.beginConnection(provider, this.callbackUri, undefined, state);
+    const url = await this.authClient.beginConnection(provider, this.callbackUri, undefined, undefined, state);
 
     const userData = await this.readUserData(userId);
     await this.writeUserData(userId, {