-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Accepting DM verification in an encrypted DM can cause unknown device warnings you can't access #11709
Comments
I was unable to repro this; is it still a valid ticket? |
depends on whether or not we are keeping the unknown devices concept at all. Currently I can reproduce it, but this was a time before the cross-signing flag. @jryans is the concept gone forever with cross signing or just part of the transition? |
The default state in a cross-signing world allows all messages, but there are further plans to support paranoia options that would block unknown devices in some way: |
action for this ticket: don't block sending for unknown devices |
if this is an issue it's extra urgent now I've landed auto-E2E for verifications 💃 |
I can't repro this anymore. I've tried adding a new device, skipping the complete-security to not have it cross-signed, and then verify that user from another account. This is with the Never send encrypted messages to unverified devices from this device flag off on the initiating side. Left is what I see from the account itself, and right from the account trying to verify it: I could successfully verify, without any unknown device warnings, and after verification completed I saw a red device for the account as well, so all seems to be working well. With the Never send encrypted messages to unverified devices from this device flag on, I get this on the receiving side (on a cross signed device): #11808 seems to be about fixing that though, so closing this issue. Looking further, it seems like matrix-org/matrix-react-sdk#3837 was a fix for this. |
This is slightly unhelpful if I have to cancel verification to verify the user...
The text was updated successfully, but these errors were encountered: