The Wayback Machine - https://web.archive.org/web/20200714061044/https://github.com/vector-im/riot-web/issues/11383
Skip to content

/ riot-web

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

Jump to bottom

SAS verification over DM doesn't work in encrypted rooms #11383

Closed
bwindels opened this issue Nov 14, 2019 · 0 comments
Closed

SAS verification over DM doesn't work in encrypted rooms #11383

bwindels opened this issue Nov 14, 2019 · 0 comments
Assignees
@bwindels
Labels
feature:e2e-cross-signing p1 phase:1 story:2
Projects
Workflow

Comments

@bwindels
Copy link
Member

@bwindels bwindels commented Nov 14, 2019 •
edited

The party that started the process gets stuck in the verification dialog waiting for the the other party to respond, even though the tile in the timeline correctly says that the other party has accepted.

Perhaps this is the verificator code in js-sdk not taking into account event decryption.
@bwindels bwindels changed the title verification over DM doesn't work in encrypted rooms SAS verification over DM doesn't work in encrypted rooms Nov 14, 2019
@bwindels bwindels added this to Raging Inferno 🔥🔥🔥 in Workflow Nov 14, 2019
@bwindels bwindels moved this from Raging Inferno 🔥🔥🔥 to Up Next in Workflow Nov 14, 2019
@bwindels bwindels moved this from Up Next to In Progress in Workflow Nov 15, 2019
@bwindels bwindels self-assigned this Nov 15, 2019
@lampholder lampholder removed the phase:1 label Nov 17, 2019
@bwindels bwindels mentioned this issue Nov 18, 2019
Merged
@bwindels bwindels moved this from In Progress to In Review in Workflow Nov 18, 2019
Workflow automation moved this from In Review to In Test Nov 19, 2019
@jryans jryans added the phase:1 label Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bwindels bwindels

Labels
feature:e2e-cross-signing p1 phase:1 story:2
Projects
Workflow
In Test
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

Fix SAS verification in encrypted DMs
3 participants
@bwindels @jryans @lampholder
You can’t perform that action at this time.