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

Resist to decryption errors in an edit chain. #7430

Closed
BillCarsonFr opened this issue Oct 21, 2022 · 2 comments
Closed

Resist to decryption errors in an edit chain. #7430

BillCarsonFr opened this issue Oct 21, 2022 · 2 comments
Labels
A-E2EE A-Message-Editing Z-UISI

Comments

@BillCarsonFr
Copy link
Member

Currently if an original message that was edited is in decryption error, the message will be displayed as a unable to decrypt/waiting for that message even if the edits are succesfully decrypted.

It's more visible recently due to this synapse bug matrix-org/synapse#14252

I think it's reasonable to display the latest content if we can decrypt it?
@brandsimon
Copy link

Workaround: long-click the message and copying it will retrieve the last edit.

@BillCarsonFr BillCarsonFr mentioned this issue Nov 16, 2022
Merged
15 tasks
@BillCarsonFr
Copy link
Member Author

Closing for now as as per spec initial event content is required for proper validation. Spec issue tracked here matrix-org/matrix-spec#1299
The change causing the issue has been reverted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-E2EE A-Message-Editing Z-UISI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@BillCarsonFr @brandsimon