@logto/react 0.1.11 → 0.1.14

package/lib/hooks/index.js

@@ -25,11 +25,12 @@ const useErrorHandler = () => {
         else if (fallbackErrorMessage) {
             setError(new Error(fallbackErrorMessage));
         }
+        console.error(error);
     }, [setError]);
     return { handleError };
 };
 const useHandleSignInCallback = (returnToPageUrl = window.location.origin) => {
-    const { logtoClient, isAuthenticated, error, setIsAuthenticated } = (0, react_1.useContext)(context_1.LogtoContext);
+    const { logtoClient, isAuthenticated, error } = (0, react_1.useContext)(context_1.LogtoContext);
     const { isLoading, setLoadingState } = useLoadingState();
     const { handleError } = useErrorHandler();
     const handleSignInCallback = (0, react_1.useCallback)(async (callbackUri) => {
@@ -39,7 +40,9 @@ const useHandleSignInCallback = (returnToPageUrl = window.location.origin) => {
         try {
             setLoadingState(true);
             await logtoClient.handleSignInCallback(callbackUri);
-            setIsAuthenticated(true);
+            // We deliberately do NOT set isAuthenticated to true here, because the app state may change immediately
+            // even before navigating to the return page URL, which might cause rendering problems.
+            // Moreover, since the location will be redirected, the isAuthenticated state will not matter any more.
             window.location.assign(returnToPageUrl);
         }
         catch (error) {
@@ -48,7 +51,7 @@ const useHandleSignInCallback = (returnToPageUrl = window.location.origin) => {
         finally {
             setLoadingState(false);
         }
-    }, [logtoClient, returnToPageUrl, setIsAuthenticated, setLoadingState, handleError]);
+    }, [logtoClient, returnToPageUrl, setLoadingState, handleError]);
     (0, react_1.useEffect)(() => {
         if (!isAuthenticated && logtoClient?.isSignInRedirected(window.location.href)) {
             void handleSignInCallback(window.location.href);
@@ -62,7 +65,7 @@ const useHandleSignInCallback = (returnToPageUrl = window.location.origin) => {
 };
 exports.useHandleSignInCallback = useHandleSignInCallback;
 const useLogto = () => {
-    const { logtoClient, loadingCount, isAuthenticated, error, setIsAuthenticated } = (0, react_1.useContext)(context_1.LogtoContext);
+    const { logtoClient, loadingCount, isAuthenticated, error } = (0, react_1.useContext)(context_1.LogtoContext);
     const { setLoadingState } = useLoadingState();
     const { handleError } = useErrorHandler();
     const isLoading = loadingCount > 0;
@@ -88,7 +91,9 @@ const useLogto = () => {
         try {
             setLoadingState(true);
             await logtoClient.signOut(postLogoutRedirectUri);
-            setIsAuthenticated(false);
+            // 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.
+            // 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.');
@@ -96,7 +101,7 @@ const useLogto = () => {
         finally {
             setLoadingState(false);
         }
-    }, [logtoClient, setIsAuthenticated, setLoadingState, handleError]);
+    }, [logtoClient, setLoadingState, handleError]);
     const fetchUserInfo = (0, react_1.useCallback)(async () => {
         if (!logtoClient) {
             return (0, context_1.throwContextError)();
@@ -134,10 +139,10 @@ const useLogto = () => {
         try {
             return logtoClient.getIdTokenClaims();
         }
-        catch (error) {
-            handleError(error, 'Unexpected error occurred while getting id token claims.');
+        catch {
+            // Do nothing if any exception occurs. Caller will get undefined value.
         }
-    }, [logtoClient, handleError]);
+    }, [logtoClient]);
     if (!logtoClient) {
         return (0, context_1.throwContextError)();
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@logto/react",
-  "version": "0.1.11",
+  "version": "0.1.14",
   "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": "fb6df238f56049607af2c3b88bee07ba2aa45dae"
+  "gitHead": "5d05c6bd23dc35d7217031edccc01db2bae33b7b"
 }