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

Trust & Decoration | Show specific decryption error for messages sent by a user in Verification Violation #3628

Closed
BillCarsonFr opened this issue Oct 8, 2024 · 3 comments · Fixed by #3729
Assignees

Comments

@BillCarsonFr
Copy link
Member

BillCarsonFr commented Oct 8, 2024

part of https://github.com/element-hq/crypto-internal/issues/308

Alternative (stop gap) for element-hq/element-meta#2523, that is the wanted final behavior where we want to display the decrypted content, alongside a warning that the sender has changed identity.

When OnlySignedDeviceIsolationMode is enabled, messages sent from insecure devices will fail to decrypt.

Currently these messages will be displayed as regular UTD:
image

We need to show a specific decryption error now.
Web went with this temporary design, see here element-hq/matrix-react-sdk#93
image

Need a design for the temporary solution where it's in UTD

Updated Design
image

@BillCarsonFr BillCarsonFr changed the title Trust & Decoration | Show specific decryption error for messages sent a user in Verification Violation Trust & Decoration | Show specific decryption error for messages sent by a user in Verification Violation Oct 8, 2024
@andybalaam
Copy link
Contributor

@BillCarsonFr will paste in the Web solution that we will copy here, and link to the final version of this task.

@BillCarsonFr
Copy link
Member Author

@BillCarsonFr will paste in the Web solution that we will copy here, and link to the final version of this task.

Done element-hq/element-meta#2523

@BillCarsonFr
Copy link
Member Author

For reference

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants