Skip to content

Fix crash when signing out.#6908

Merged
bmarty merged 1 commit intodevelopfrom
feature/bma/fix_crash
Aug 23, 2022
Merged

Fix crash when signing out.#6908
bmarty merged 1 commit intodevelopfrom
feature/bma/fix_crash

Conversation

@bmarty
Copy link
Member

@bmarty bmarty commented Aug 22, 2022

At this point, we do not want the sync to start, the DB is already deleted.

Type of change

  • Feature
  • Bugfix
  • Technical
  • Other :

Content

Do not start the SyncThread if the user want to sign out.
Related to #6884

Motivation and context

Fix crash when signing out.

Screenshots / GIFs

Tests

  • Step 1
  • Step 2
  • Step ...

Tested devices

  • Physical
  • Emulator
  • OS version(s):

Checklist

At this point, we do not want the sync to start, the DB is already deleted.
@bmarty bmarty added the Z-NextRelease For issues and PRs which should be included in the NextRelease. label Aug 22, 2022
@bmarty bmarty requested a review from ouchadam August 22, 2022 14:20
Copy link
Contributor

@ouchadam ouchadam left a comment

Choose a reason for hiding this comment

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

makes sense 👍

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@bmarty
Copy link
Member Author

bmarty commented Aug 22, 2022

A test is failing (twice), first time I see that, so maybe related to the change in this PR :/

org.matrix.android.sdk.internal.crypto.E2eeSanityTests > testBasicBackupImport[test(AVD) - 9] FAILED
java.lang.IllegalStateException: This Realm instance has already been closed, making it unusable.

@bmarty
Copy link
Member Author

bmarty commented Aug 23, 2022

#6917 also suffers from the same test error. We will investigate later.

@bmarty bmarty merged commit 1c8fcca into develop Aug 23, 2022
@bmarty bmarty deleted the feature/bma/fix_crash branch August 23, 2022 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Z-NextRelease For issues and PRs which should be included in the NextRelease.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants