@logto/react 0.1.12 → 0.1.15

package/lib/hooks/index.d.ts

@@ -9,7 +9,7 @@ declare type Logto = {
     signIn: (redirectUri: string) => Promise<void>;
     signOut: (postLogoutRedirectUri: string) => Promise<void>;
 };
-declare const useHandleSignInCallback: (returnToPageUrl?: string) => {
+declare const useHandleSignInCallback: (callback?: (() => void) | undefined) => {
     isLoading: boolean;
     isAuthenticated: boolean;
     error: Error | undefined;

package/lib/hooks/index.js

@@ -29,10 +29,15 @@ const useErrorHandler = () => {
     }, [setError]);
     return { handleError };
 };
-const useHandleSignInCallback = (returnToPageUrl = window.location.origin) => {
+const useHandleSignInCallback = (callback) => {
     const { logtoClient, isAuthenticated, error, setIsAuthenticated } = (0, react_1.useContext)(context_1.LogtoContext);
     const { isLoading, setLoadingState } = useLoadingState();
     const { handleError } = useErrorHandler();
+    const callbackRef = (0, react_1.useRef)();
+    (0, react_1.useEffect)(() => {
+        // eslint-disable-next-line @silverhand/fp/no-mutation
+        callbackRef.current = callback; // Update ref to the latest callback.
+    }, [callback]);
     const handleSignInCallback = (0, react_1.useCallback)(async (callbackUri) => {
         if (!logtoClient) {
             return (0, context_1.throwContextError)();
@@ -41,7 +46,7 @@ const useHandleSignInCallback = (returnToPageUrl = window.location.origin) => {
             setLoadingState(true);
             await logtoClient.handleSignInCallback(callbackUri);
             setIsAuthenticated(true);
-            window.location.assign(returnToPageUrl);
+            callbackRef.current?.();
         }
         catch (error) {
             handleError(error, 'Unexpected error occurred while handling sign in callback.');
@@ -49,10 +54,11 @@ const useHandleSignInCallback = (returnToPageUrl = window.location.origin) => {
         finally {
             setLoadingState(false);
         }
-    }, [logtoClient, returnToPageUrl, setIsAuthenticated, setLoadingState, handleError]);
+    }, [logtoClient, setLoadingState, setIsAuthenticated, handleError]);
     (0, react_1.useEffect)(() => {
-        if (!isAuthenticated && logtoClient?.isSignInRedirected(window.location.href)) {
-            void handleSignInCallback(window.location.href);
+        const currentPageUrl = window.location.href;
+        if (!isAuthenticated && logtoClient?.isSignInRedirected(currentPageUrl)) {
+            void handleSignInCallback(currentPageUrl);
         }
     }, [handleSignInCallback, isAuthenticated, logtoClient]);
     return {
@@ -91,7 +97,7 @@ const useLogto = () => {
             await logtoClient.signOut(postLogoutRedirectUri);
             // We deliberately do NOT set isAuthenticated to false here, because the app state may change immediately
             // even before navigating to the oidc end session endpoint, which might cause rendering problems.
-            // Instead, we will reload isAuthenticated state when the user is redirected back to the app.
+            // Moreover, since the location will be redirected, the isAuthenticated state will not matter any more.
         }
         catch (error) {
             handleError(error, 'Unexpected error occurred while signing out.');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@logto/react",
-  "version": "0.1.12",
+  "version": "0.1.15",
   "main": "./lib/index.js",
   "exports": "./lib/index.js",
   "typings": "./lib/index.d.ts",
@@ -24,7 +24,7 @@
     "prepack": "pnpm test"
   },
   "dependencies": {
-    "@logto/browser": "^0.1.11",
+    "@logto/browser": "^0.1.14",
     "@silverhand/essentials": "^1.1.6"
   },
   "devDependencies": {
@@ -38,7 +38,7 @@
     "@types/react": "^17.0.39",
     "eslint": "^8.9.0",
     "jest": "^27.5.1",
-    "lint-staged": "^12.3.4",
+    "lint-staged": "^13.0.0",
     "postcss": "^8.4.6",
     "prettier": "^2.5.1",
     "react": "^17.0.2",
@@ -56,5 +56,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "1e7449daf62edac7f7dd2eb37f3e5b83e6bf3c56"
+  "gitHead": "51665f47934ef390136ee8ff53393f20f9fb8221"
 }