Skip to content

Dedicated API and UI for verification failure in protected groups #4999

Open
@link2xt

Description

@link2xt

Historically we replaced the message with an error in square brackets in two cases:

  • [Unknown sender for this chat. See 'info' for more details.]
  • [Verification problem: {err} See 'Info' for more details]

This is a hacky way to display an error as it prevent download of images.
It is also known to interact badly with reaction processing, if reaction message is replaced with an error message this happens:

Screenshots

"Unknown sender for this chat" error happens when in a protected chat a message is received from someone who is not a member of the group. This has been replaced with a normal error in #5024 and fixed a common case of late messages arriving after member removal #4852.

There was also a PR #4988 that simply replaced all square bracket errors with normal ones, but it is closed without merging.

Remaining issue is about the "verification problem" error when a message is not signed with the sender verified key in a protected group.

Signal at least at some point had this "Tap to process and display", something like an undownloaded message:
sn-advisory-receive
(from https://signal.org/blog/verified-safety-number-updates/)

Ideally unverified message should be an error type like this, so the message is not displayed immediately but can be revealed and then hidden once you reopen the chat. So properly fixing it will need UI work to hide messages by default and show them when user has read a warning and clicks a button to reveal the message contents.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions