Skip to content

Commit

Permalink
Merge pull request #38503 from DylanDylann/fix/38163
Browse files Browse the repository at this point in the history
fix: annonymous user can open setting
  • Loading branch information
madmax330 authored Mar 25, 2024
2 parents eba217f + c54a980 commit 35a78d6
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions src/libs/actions/Link.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ import CONST from '@src/CONST';
import ONYXKEYS from '@src/ONYXKEYS';
import type {Route} from '@src/ROUTES';
import ROUTES from '@src/ROUTES';
import * as Session from './Session';

let isNetworkOffline = false;
Onyx.connect({
Expand Down Expand Up @@ -102,6 +103,10 @@ function openLink(href: string, environmentURL: string, isAttachment = false) {
// If we are handling a New Expensify link then we will assume this should be opened by the app internally. This ensures that the links are opened internally via react-navigation
// instead of in a new tab or with a page refresh (which is the default behavior of an anchor tag)
if (internalNewExpensifyPath && hasSameOrigin) {
if (Session.isAnonymousUser() && !Session.canAnonymousUserAccessRoute(internalNewExpensifyPath)) {
Session.signOutAndRedirectToSignIn();
return;
}
Navigation.navigate(internalNewExpensifyPath as Route);
return;
}
Expand Down

0 comments on commit 35a78d6

Please sign in to comment.