-
Notifications
You must be signed in to change notification settings - Fork 274
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
User verification via room might fail depending on the encryption settings #4768
Comments
We think this will be fixed by "Exclude Insecure Devices" - we should check that assumption. |
Andreas will check this. |
Tested and can confirm that if before doing this Alice turns ON the feature flag to exclude insecure devices, the user verification works. |
Might fix this specific case, but there are probably other cases. For example if the other user is in verification violation, there are 2 options to restore communication:
But in that case |
That's a valid point, we do want to allow re-verify without requiring withdrawing verification. We then have to do the same fix as for the legacy apps which is to send such messages even when the device is insecure. |
Step to reproduce:
Actual:
Nothing happens
Expected:
Verification should start
The error is:
As per spec:
A similar "fix" was done on legacy sdks, to "allow" sending verification messages even to insecure devices
The text was updated successfully, but these errors were encountered: