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

fix: MLS account not being deleted when logging out - WPB-15774 #2572

Merged
merged 2 commits into from
Feb 21, 2025

Conversation

jullianm
Copy link
Contributor

@jullianm jullianm commented Feb 20, 2025

BugWPB-15774 [iOS] Logging out consistently doesn't work (Canary)

Issue

Context: When logging out from MLS account (userA) and logging in with other account (userB) and logging out again it will results in weird behavior (crashes, session corrupted popup or login screen showing expired session message..)
-> app behaves inconsistently when logging out from MLS account.

Causes: When logging out from MLS account, we're deleting everything related to the active session except the account itself because some completion block is not called whereas it should be - whether it is a MLS or Proteus account (works fine if we only test with Proteus accounts).

Solution: Ensure the code responsible to delete account data is properly called when logging out from a MLS account by removing usage of dispatch group.

Testing

Log in with MLS account
Log out
Log in with other account (MLS or Proteus)
Log out
..
-> app should behave consistently when logging in / out from MLS or Proteus accounts (no corrupted session pop up, no crash, no expired session message on login screen..)

Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

@jullianm jullianm marked this pull request as ready for review February 21, 2025 13:55
@jullianm jullianm requested review from netbe and caldrian February 21, 2025 13:55
Copy link
Collaborator

@netbe netbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

Copy link
Contributor

Test Results

1 314 tests   1 312 ✅  2m 5s ⏱️
    1 suites      2 💤
    1 files        0 ❌

Results for commit 6483fb3.

@datadog-wireapp
Copy link

Datadog Report

Branch report: fix/mls-log-out-error-other-fix
Commit report: 6f74932
Test service: wire-ios-mono

✅ 0 Failed, 1294 Passed, 2 Skipped, 2m 5.83s Total Time

@jullianm jullianm merged commit 46cc050 into release/cycle-3.118 Feb 21, 2025
12 of 13 checks passed
@jullianm jullianm deleted the fix/mls-log-out-error-other-fix branch February 21, 2025 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants