Skip to content

Sign in – Unable to login via Magic link, Magic code expired appears #48378

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

Open
2 of 6 tasks
IuliiaHerets opened this issue Aug 31, 2024 · 60 comments
Open
2 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review Weekly KSv2

Comments

@IuliiaHerets
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: 9.0.27-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4906568
Email or phone of affected tester (no customers): [email protected]
Issue reported by: Applause Internal Team

Action Performed:

  1. Go to https://staging.new.expensify.com/
  2. In Sign in page enter new Expensify email
  3. Open email and copy Magic link
  4. Change it to staging and navigate to it in a new tab

Expected Result:

Abracadabra page opens. We have KI #44600

Actual Result:

Magic code expired appears

Workaround:

Unknown

Platforms:

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

Screenshots/Videos

Bug6588552_1725081784278.magic_link.mp4

View all open jobs on GitHub

@IuliiaHerets IuliiaHerets added DeployBlockerCash This issue or pull request should block deployment Bug Something is broken. Auto assigns a BugZero manager. DeployBlocker Indicates it should block deploying the API labels Aug 31, 2024
Copy link

melvin-bot bot commented Aug 31, 2024

Triggered auto assignment to @muttmuure (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

Copy link

melvin-bot bot commented Aug 31, 2024

Triggered auto assignment to @danieldoglas (DeployBlockerCash), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

@github-actions github-actions bot added Engineering Hourly KSv2 and removed Daily KSv2 labels Aug 31, 2024
Copy link
Contributor

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

@ishpaul777
Copy link
Contributor

Can't reproduce, tried multiple times

Screen.Recording.2024-09-02.at.2.10.02.AM.mov

@danieldoglas
Copy link
Contributor

@IuliiaHerets can you try this one again? Looks like you were logged in on a tab already on your video, so maybe that's why?

@danieldoglas danieldoglas removed the DeployBlocker Indicates it should block deploying the API label Sep 2, 2024
@danieldoglas
Copy link
Contributor

not a deploy blocker for the API since it doesn't happen in production

@danieldoglas danieldoglas added Weekly KSv2 retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause and removed DeployBlockerCash This issue or pull request should block deployment Hourly KSv2 labels Sep 2, 2024
@IuliiaHerets
Copy link
Author

@danieldoglas Issue is still reproducible, build - v9.0.27-0

Screenrecorder-2024-09-02-20-39-56-247.mp4

@ishpaul777
Copy link
Contributor

We are awaiting proposals

@danieldoglas
Copy link
Contributor

@ishpaul777 are you able to reproduce this now?

@ishpaul777
Copy link
Contributor

ishpaul777 commented Sep 4, 2024

Lol no, my instinct kicked in and i wrote "awaiting proposal", what i mean is waiting reliable steps to repro

Screen.Recording.2024-09-05.at.1.38.57.AM.mov

@deetergp
Copy link
Contributor

deetergp commented Sep 5, 2024

I just ran into this on staging by sending an Invoice from one user to an email address that does not yet have an Expensify account. I was able to reproduce a second time by sending to another email without an account.

@danieldoglas
Copy link
Contributor

nice, thanks for that @deetergp . @ishpaul777 can you try those steps and confirm it still fails? Then we can open it for external proposals.

@ishpaul777
Copy link
Contributor

Seems we have a existing issue that we can not send a invoice to any user.

https://expensify.slack.com/archives/C01GTK53T8Q/p1725613866403789

@eVoloshchak
Copy link
Contributor

eVoloshchak commented Mar 5, 2025

@danieldoglas, according to this thread I'll be taking this over as C+ from @ishpaul777

Could you re-assign this please?

Is the backend not returning something, or is it something in the app?

I haven't looked into this too closely, could be either of those just by looking at the code, but I suspect this is a back-end issue. Let's not make it External just yet, I'll look into this closer and post my findings tomorrow

@danieldoglas
Copy link
Contributor

Done, assigning @eVoloshchak to it!

@izarutskaya
Copy link

Hi @CortneyOfstad No adjustment to the reproducible steps. It is stell reproducible according steps. Build v9.1.14-4

Recording.3017.mp4

@eVoloshchak
Copy link
Contributor

It's not clear whether it's purely a back-end issue, should we open this for proposals?

@danieldoglas
Copy link
Contributor

@eVoloshchak you said you were going to post your findings, what were those?

@eVoloshchak
Copy link
Contributor

Wasn't able to find anything, we display the Magic Code Expired modal if auth session state is failed (which it is in this case).
But it's not clear why authentication is failing (is this a back-end issue with not being able to sign in accounts that are not yet verified or is it something about the new accounts that we're handling incorrectly on the front end)

@CortneyOfstad
Copy link
Contributor

Hey @danieldoglas @eVoloshchak! I am going to be heading OoO this afternoon so if this needs BZ action, please re-assign the label. Otherwise I will review when I get back!

@CortneyOfstad
Copy link
Contributor

Back from OoO — @eVoloshchak and @danieldoglas — where are we at with this?

@danieldoglas
Copy link
Contributor

bump @eVoloshchak . Anything else you could find?

@mvtglobally
Copy link

Issue is reproducible during KI retests.

Unable to login via Magic link, Magic code expired appears

Image

@mvtglobally mvtglobally removed the retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause label Apr 19, 2025
@eVoloshchak
Copy link
Contributor

@danieldoglas, traced this back to SignInUserWithLink API request returning an error

Image

We do send the correct account ID and validation code, should be working

Image

This is probably a BE issue, I suggest we make this Internal

@danieldoglas
Copy link
Contributor

Interesting - thanks for sending the requestID, that was very helpful.

I can see that we received the validate code correctly, but looks like it didn't match the one we had available in the DB. I'll investigate more.

@CortneyOfstad
Copy link
Contributor

Thanks @danieldoglas!

@CortneyOfstad
Copy link
Contributor

Hey @danieldoglas — just checking in to see if there were any updates on this. Thanks!

@danieldoglas danieldoglas added Weekly KSv2 and removed Daily KSv2 labels Apr 28, 2025
@danieldoglas
Copy link
Contributor

I think I know what's kind of going on here, but I'll need some time to discuss. Changing this to weekly while I finish some other pressing matters, but it's on my radar.

@CortneyOfstad
Copy link
Contributor

Sounds good — thanks @danieldoglas!

@CortneyOfstad
Copy link
Contributor

Hey @danieldoglas — just checking to see if we have a rough ETA when this will be looked at? Thanks!

@CortneyOfstad
Copy link
Contributor

Bump @danieldoglas ^^^ thanks!

@danieldoglas
Copy link
Contributor

Still on my list, but not a huge priority compared to some other tasks.

I'm going to try to prioritize this in the next week

@CortneyOfstad
Copy link
Contributor

Sounds good @danieldoglas — thank you!

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. Engineering Internal Requires API changes or must be handled by Expensify staff Reviewing Has a PR in review Weekly KSv2
Projects
Status: No status
Development

No branches or pull requests