The Wayback Machine - https://web.archive.org/web/20200714061628/https://github.com/vector-im/riot-web/issues/11968
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

check in SSSS if user is really verified when showing 'X is verified' tile #11968

Closed
bwindels opened this issue Jan 21, 2020 · 1 comment
Closed

check in SSSS if user is really verified when showing 'X is verified' tile #11968

bwindels opened this issue Jan 21, 2020 · 1 comment
Assignees
@foldleft
Labels
feature:e2e-cross-signing phase:1 story:2
Projects
Bruno's short term TODO list Workflow

Comments

@bwindels
Copy link
Member

@bwindels bwindels commented Jan 21, 2020 •
edited

in MKeyVerificationConclusion, check with the crypto layer if this user is still/really verified, and if not, just return null I guess to show a blank tile?
@bwindels bwindels added story:35 story:2 and removed story:35 labels Jan 21, 2020
@bwindels
Copy link
Member Author

@bwindels bwindels commented Jan 31, 2020

I think this might just be cli.checkUserTrust(userId).isCrossSigningVerified() but would need to double check with ryan or dave.
@foldleft foldleft self-assigned this Jan 31, 2020
@foldleft foldleft added this to In Progress in Workflow via automation Jan 31, 2020
@foldleft foldleft mentioned this issue Jan 31, 2020
Merged
@foldleft foldleft moved this from In Progress to In Review in Workflow Jan 31, 2020
Workflow automation moved this from In Review to In Test Feb 4, 2020
@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

@foldleft foldleft

Labels
feature:e2e-cross-signing phase:1 story:2
Projects
Workflow
In Test
Bruno's short term TODO list
TODO (hi risk tasks first)
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

double-check user verification
3 participants
@bwindels @jryans @foldleft
You can’t perform that action at this time.