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

Verify device has no effect #2132

Closed
manuroe opened this issue Dec 14, 2018 · 1 comment
Closed

Verify device has no effect #2132

manuroe opened this issue Dec 14, 2018 · 1 comment
Assignees
Labels
A-E2EE P1 T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Milestone

Comments

@manuroe
Copy link
Member

manuroe commented Dec 14, 2018

Steps to reproduce:

  • in the timeline of an e2e room, tap on the warning triangle on a message from a non verified device
  • verify that device

-> 80% of the time, the warning triangle is still displayed, meaning the device has not been verified.
The issue persists after an app restart

@manuroe manuroe added T-Defect Something isn't working: bugs, crashes, hangs and other reported problems P1 A-E2EE labels Dec 14, 2018
@manuroe manuroe added this to the Sprint 16 milestone Dec 14, 2018
@manuroe manuroe self-assigned this Dec 17, 2018
@manuroe
Copy link
Member Author

manuroe commented Dec 17, 2018

The bug comes from the introduction and the usage of [MXCryptoStore deviceWithIdentityKey:] method in matrix-org/matrix-ios-sdk@481623f.

This method is not buggy but it reveals that the way we store user devices is wrong and leads to device duplicates. This bug is at https://github.com/matrix-org/matrix-ios-sdk/blob/40989276f4519e157bdcb04589ef00fdf3fdb12b/MatrixSDK/Crypto/Data/Store/MXRealmCryptoStore/MXRealmCryptoStore.m#L479.

This bug was hidden and it worked fine when there is only one device getter method in the crypto store. With the new [MXCryptoStore deviceWithIdentityKey:], we make a different query that gets a different object but with the same data :/.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-E2EE P1 T-Defect Something isn't working: bugs, crashes, hangs and other reported problems
Projects
None yet
Development

No branches or pull requests

1 participant