From 9c13bd65df26e8e9bcb0b0c72b68adad45134fc2 Mon Sep 17 00:00:00 2001
From: Sam <sam@lowdefy.com>
Date: Thu, 23 Jun 2022 16:50:55 +0200
Subject: [PATCH] feat: Add callbackUrl and redirect as logout action params.

---
 packages/client/src/auth/createAuthMethods.js | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/packages/client/src/auth/createAuthMethods.js b/packages/client/src/auth/createAuthMethods.js
index fb1077734..677905578 100644
--- a/packages/client/src/auth/createAuthMethods.js
+++ b/packages/client/src/auth/createAuthMethods.js
@@ -37,7 +37,7 @@ function getCallbackUrl({ lowdefy, callbackUrl = {} }) {
 function createAuthMethods({ lowdefy, auth }) {
   // login and logout are Lowdefy function that handle action params
   // signIn and signOut are the next-auth methods
-  function login({ providerId, callbackUrl, authUrl = {} } = {}) {
+  function login({ authUrl, callbackUrl, providerId } = {}) {
     if (type.isNone(providerId) && auth.authConfig.providers.length === 1) {
       providerId = auth.authConfig.providers[0].id;
     }
@@ -45,12 +45,11 @@ function createAuthMethods({ lowdefy, auth }) {
     auth.signIn(
       providerId,
       { callbackUrl: getCallbackUrl({ lowdefy, callbackUrl }) },
-      authUrl.urlQuery
+      authUrl?.urlQuery
     );
   }
-  // TODO: fix callbackUrl
-  function logout() {
-    auth.signOut();
+  function logout({ callbackUrl, redirect } = {}) {
+    auth.signOut({ callbackUrl: getCallbackUrl({ lowdefy, callbackUrl }), redirect });
   }
   return {
     login,