fix(sign-in): Fix redirect correct org after accepting invite

[priscilawebdev]

closes #43745
closes #67690

[codecov]

❌ 2 Tests Failed:

Tests completed	Failed	Passed	Skipped
23458	2	23456	272

View the top 1 failed tests by shortest run time

tests.acceptance.test_accept_organization_invite.AcceptOrganizationInviteTest test_existing_user_invite_2fa_enforced_org

Stack Traces | 0.088s run time

#x1B[1m#x1B[.../sentry/testutils/cases.py#x1B[0m:1044: in tearDown
    self.wait_for_loading()
#x1B[1m#x1B[.../sentry/testutils/cases.py#x1B[0m:1037: in wait_for_loading
    self.browser.wait_until_not('[data-test-id="loading-indicator"]')
#x1B[1m#x1B[.../testutils/pytest/selenium.py#x1B[0m:250: in wait_until_not
    wait.until_not(
#x1B[1m#x1B[31m.venv/lib/python3.12.../webdriver/support/wait.py#x1B[0m:128: in until_not
    raise TimeoutException(message)
#x1B[1m#x1B[31mE   selenium.common.exceptions.TimeoutException: Message:#x1B[0m

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

[markstory]

Doesn't this bypass the membership invite acceptance views? We would also be skipping important steps of that process like enforcing MFA and SSO.

[priscilawebdev]

Doesn't this bypass the membership invite acceptance views?

No, it doesn't. If the user is not logged in and clicks on the invitation, they will first see the invitation view, where they can either "Create a new account" or "Login using an existing account."

Currently, after submitting the form to create a new account, the invitation is accepted, and the user is redirected to the organization associated with the invite. However, this doesn't happen when the user chooses to login using an existing account. This is what this PR is trying to fix

[priscilawebdev]

We would also be skipping important steps of that process like enforcing MFA and SSO.

I’ve moved the code to what I believe is a more appropriate location, and I was wondering if we have existing tests for the use cases you mentioned. The only tests I could find are these, which are still passing.

Could you confirm if these cover the scenarios we need?

[mifu67]

I just pulled your branch and tested things out. I think that we do bypass some of the member invite acceptance views by taking the user directly to the org. What I think should happen instead is that after the user logs in, they are redirected back to the initial member invite screen, where they have an active session and can accept the invitation with their currently signed-in account.

We're planning to implement this new member invite flow as part of our upcoming refactor of the authentication system.

[markstory]

The scenario I'm thinking of is:

1. An existing sentry user is invited to a new org.
2. The user visits the organization login page
3. If the user logs in, they'll implicitly accept the membership invite (via these changes) and bypass SSO setup and MFA setup requirements to gain membership.

[getsantry]

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you add the label WIP, I will leave it alone unless WIP is removed ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀