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.

Sign up for GitHub

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

Jump to bottom

[Rooms list] Last messages are stuck encrypted whereas the related keys have been received #4867

Closed
giomfo opened this issue Jan 5, 2022 · 2 comments · Fixed by #5080
Closed

[Rooms list] Last messages are stuck encrypted whereas the related keys have been received #4867

giomfo opened this issue Jan 5, 2022 · 2 comments · Fixed by #5080
Assignees
@ouchadam
Labels
A-E2EE O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Papercuts Visible. Impactful. Predictable to action.

Comments

@giomfo
Copy link
Member

giomfo commented Jan 5, 2022

Steps to reproduce

  1. Log in with an account that is logged in on another session. The last message of the most recent encrypted rooms is decrypted on this existing session
  2. Verify the new session with the existing one
  3. The rooms list is not updated whereas the keys are received, all the last messages are displayed encrypted. If we open one of the room the messages are decrypted and when we go back to the rooms list the last message is displayed correctly

Outcome

This sounds like the rooms list is not updated whereas the room summaries have been updated

Your phone model

No response

Operating system version

No response

Application version and app store

No response

Homeserver

No response

Will you send logs?

No
@giomfo giomfo added the T-Defect Something isn't working: bugs, crashes, hangs and other reported problems label Jan 5, 2022
@giomfo giomfo mentioned this issue Jan 5, 2022
Closed
@Claire1817
Copy link
Contributor

Claire1817 commented Jan 20, 2022 •
edited
Loading

We should add a listener when there is a new trusted session.
This listener will update the latestPreviewableEvent in the RoomSummary if the key could decrypt it.

@ouchadam ouchadam added A-E2EE O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist Z-Papercuts Visible. Impactful. Predictable to action. labels Jan 24, 2022
@ouchadam ouchadam added S-Minor Impairs non-critical functionality or suitable workarounds exist and removed S-Minor Impairs non-critical functionality or suitable workarounds exist labels Jan 24, 2022
@ouchadam ouchadam self-assigned this Jan 26, 2022
@ouchadam ouchadam mentioned this issue Jan 27, 2022
Merged
@ouchadam
Copy link
Contributor

Fixed in 1.4.10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ouchadam ouchadam

Labels
A-E2EE O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect Something isn't working: bugs, crashes, hangs and other reported problems Z-Papercuts Visible. Impactful. Predictable to action.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Updating encrypted room list message previews on key updates element-hq/element-android
3 participants
@ouchadam @giomfo @Claire1817