fix(rbac): added type check for userRole

Author: imjulianeral

example/ts-fire-example/package-lock.json

@@ -11,7 +11,7 @@
   "dependencies": {
     "firebase": "^8.7.0",
     "next": "11.0.1",
-    "next-shield": "^2.0.1",
+    "next-shield": "^2.0.2",
     "react": "17.0.2",
     "react-dom": "17.0.2",
     "react-firebase-hooks": "^3.0.4",

example/ts-fire-example/package.json

@@ -63,19 +63,23 @@ export function NextShield<
   const pathIsHybrid = verifyPath(hybridRoutes, pathname)
   const pathIsAuthorized =
     RBAC && userRole && verifyPath(RBAC[userRole].grantedRoutes, pathname)
-  const route = getAccessRoute(RBAC, userRole, accessRoute, loginRoute)
+  const access = getAccessRoute(RBAC, userRole, accessRoute, loginRoute)
 
   useEffect(() => {
     if (!isAuth && !isLoading && pathIsPrivate) replace(loginRoute)
-    if (isAuth && !isLoading && pathIsPublic)
-      replace(route)
-
-    if (isAuth && userRole && !isLoading && !pathIsHybrid && !pathIsAuthorized)
-      replace(route)
+    if (isAuth && !isLoading && pathIsPublic) replace(access)
+    if (
+      isAuth &&
+      typeof userRole === 'string' &&
+      !isLoading &&
+      !pathIsHybrid &&
+      !pathIsAuthorized
+    )
+      replace(access)
   }, [
     replace,
     userRole,
-    route,
+    access,
     isAuth,
     isLoading,
     loginRoute,