-
Notifications
You must be signed in to change notification settings - Fork 98
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
Room previews show redacted messages #1441
Comments
Given my limited knowledge of sliding sync and the fact that these messages reappear after logging out, this might well be a bug in the ss proxy (v0.99.5) and not ElX. Feel free to close this if this is the wrong project to report this in. |
I can reliably see the latest redacted message in an unencrypted room in the preview on a freshly install Element X 1.2.6. If I am not completely mistaken about how redactions work, this suggest this is not (only) a bug in local state management but (also) a problem in the ss proxy since the data has to come from somewhere. Should I open a separate issue over there? |
Yes please! I think https://github.com/matrix-org/sliding-sync/ is the correct place. |
Server-side fixes are in place, not yet deployed on matrix.org. Still requires client-side fixes to retroactively remove the redacted event from the room list preview. |
I'm pretty confident that this is mostly fixed with the server bugfix. The only remaining bug should be that when the server sends a non-redacted event and its redaction in the same response, we fail to honor the redaction. I don't think that remaining bug should have high priority (even though it's clearly a bug). |
an edge case where we leak redacted data just because the sync response included both redacted and unredacted data feels like it should still be high priority, imo. we should always honour redactions. |
I am considering the issue fixed and closing it. A fresh installation of EX using a server with the bug fix will not have the issue. The issue will not automatically fix itself after the server updates but this is not something we want to target. The last message will be redacted if the user opens the room or makes a clear cache. This is enough. |
Steps to reproduce
Message
, open a different room and go back to the room list again)Outcome
What did you expect?
Show an indicator a message was removed like in Element (“Message deleted”) or the previous message or something other than the thing that was supposed to be removed.
What happened instead?
Content of the redacted message is shown.
The message also reappears after a logout/login for unencrypted rooms and shows as “Message” for encrypted ones (probably only because of missing keys)
Your phone model
iPhone 13 mini / iPad Pro
Operating system version
iOS 17 Beta / iPadOS 16.5.1
Application version
1.2.1 (62)
Homeserver
Synapse 1.89.0
Will you send logs?
No
The text was updated successfully, but these errors were encountered: