Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.57k stars 2.91k forks source link

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

Open IuliiaHerets opened 2 months ago

IuliiaHerets commented 2 months ago

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com 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): applausetester+jp_330824@applause.expensifail.com 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 https://github.com/Expensify/App/issues/44600

Actual Result:

Magic code expired appears

Workaround:

Unknown

Platforms:

Screenshots/Videos

https://github.com/user-attachments/assets/95bd518d-9518-4026-93a7-72a999342cea

View all open jobs on GitHub

melvin-bot[bot] commented 2 months ago

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.

melvin-bot[bot] commented 2 months ago

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

github-actions[bot] commented 2 months ago

:wave: 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 commented 2 months ago

Can't reproduce, tried multiple times

https://github.com/user-attachments/assets/89c82774-0c55-4004-8db4-32d54199f9e7

danieldoglas commented 2 months ago

@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 commented 2 months ago

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

IuliiaHerets commented 2 months ago

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

https://github.com/user-attachments/assets/257f319c-757c-4f47-86e3-5ca4c612d231

ishpaul777 commented 2 months ago

We are awaiting proposals

danieldoglas commented 2 months ago

@ishpaul777 are you able to reproduce this now?

ishpaul777 commented 2 months ago

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

https://github.com/user-attachments/assets/ba5d5188-4358-4c4a-a71a-abd4150b0c43

deetergp commented 2 months ago

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 commented 2 months ago

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 commented 2 months ago

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

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

melvin-bot[bot] commented 2 months ago

@danieldoglas @muttmuure @ishpaul777 this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

ishpaul777 commented 2 months ago

still not able to reproduce the issue, i see theres draft PR that may fix this issue https://github.com/Expensify/App/pull/48899

ishpaul777 commented 1 month ago

Same as ^

danieldoglas commented 1 month ago

K, I'll close this issue for now.

IuliiaHerets commented 1 month ago

@danieldoglas QA team still can repro the issue, build - v9.0.46-2

https://github.com/user-attachments/assets/c14af83f-ad55-4fa9-8c2b-73de3b29cee7

danieldoglas commented 1 month ago

can you try to not delete the https:// part of the URL? I think the issue is that there's a redirect happening from HTTP > https and the router is somehow not figuring out the whole path string.

It works normally for me in the same scenario:

https://github.com/user-attachments/assets/a0b08175-f9a6-49ee-b6a5-e18a6e1f090b

melvin-bot[bot] commented 1 week ago

This issue has not been updated in over 15 days. @danieldoglas, @muttmuure, @ishpaul777 eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

ishpaul777 commented 1 week ago

@IuliiaHerets please help retest, i can't reproduce

IuliiaHerets commented 1 week ago

@ishpaul777 Tester can't reproduce this issue in the latest build (v9.0.60-0)

https://github.com/user-attachments/assets/70ac4da6-803f-4728-9dfa-eec81d5b0685

ishpaul777 commented 1 week ago

@muttmuure/ @danieldoglas Should we close this ?

danieldoglas commented 1 week ago

Yep, closing!

IuliiaHerets commented 4 days ago

QA team can reproduce this issue in the latest build (9.0.64-0), issue still repro using this comment too

https://github.com/user-attachments/assets/a75074a2-3621-4c65-b3b0-6366b7e26501

ishpaul777 commented 3 days ago

🤦 i'll advertise this on slack

CyberAndrii commented 3 days ago

BE issue. SigninUserWithLink returns the following error for unverified accounts. Verified accounts work without issues.

{
    "code": 666,
    "jsonCode": 666,
    "type": "Expensify\\Libs\\Error\\ExpError",
    "UUID": "58c55da5-23a5-4926-84d8-7c0319e16e91",
    "message": "Invalid request. Please try signing again with link from the email",
    "title": "",
    "data": [],
    "htmlMessage": "",
    "requestID": "8e523d3feef83542-WAW"
}