Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Login-Signing out & refreshing the login page, user is logged in again #36675

Closed
3 of 6 tasks
izarutskaya opened this issue Feb 16, 2024 · 9 comments
Closed
3 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Needs Reproduction Reproducible steps needed retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Weekly KSv2

Comments

@izarutskaya
Copy link

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: 1.4.42-0
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation:

Action Performed:

  1. Go to https://staging.new.expensify.com/
  2. Using "G", login as gmail user
  3. Tap profile icon
  4. Tap sign out
  5. Now refresh the login page immediately

Expected Result:

Signing out and refreshing the login page, user must not be logged in again.

Actual Result:

Signing out and refreshing the login page, user is logged in again.

The app with infinite loading home page is shown instead of login page on refresh.

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6381583_1708067209678.gs.mp4

View all open jobs on GitHub

@izarutskaya izarutskaya added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Feb 16, 2024
Copy link

melvin-bot bot commented Feb 16, 2024

Triggered auto assignment to @stephanieelliott (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@izarutskaya
Copy link
Author

I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

@izarutskaya
Copy link
Author

We think that this bug might be related to #wave9-collect-signup
CC @MitchExpensify

@MitchExpensify
Copy link
Contributor

Any ideas on this @francoisl ? Happy to loop into Wave 9

@francoisl
Copy link
Contributor

Hm at first glance I don't think it's related to wave9. @hayata-suenaga is this the same thing you were looking into already and asking about here?

@hayata-suenaga
Copy link
Contributor

No I don't believe this is related to the issue I'm working on

I'm working on an issue where the user is logged out when they're not supposed

The issue described in this issue is happening probably because we're not clearing the auth cookie

@melvin-bot melvin-bot bot added the Overdue label Feb 19, 2024
Copy link

melvin-bot bot commented Feb 19, 2024

@stephanieelliott Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@stephanieelliott
Copy link
Contributor

Hm, I'm also not able to repro this. Gonna apply the label to have QA include this in the retest rotation, but if we can't repro it and we it's not tied to a wave perhaps we should close.

@melvin-bot melvin-bot bot removed the Overdue label Feb 19, 2024
@stephanieelliott stephanieelliott added Needs Reproduction Reproducible steps needed Overdue retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Weekly KSv2 and removed Overdue Daily KSv2 labels Feb 19, 2024
@mvtglobally
Copy link

Issue not reproducible during KI retests. (First week)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Needs Reproduction Reproducible steps needed retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Weekly KSv2
Projects
None yet
Development

No branches or pull requests

6 participants