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

Remove keyshare dialog once cross-signing exits labs #11908

Closed
jryans opened this issue Jan 17, 2020 · 4 comments
Closed

Remove keyshare dialog once cross-signing exits labs #11908

jryans opened this issue Jan 17, 2020 · 4 comments

Comments

@jryans
Copy link
Member

@jryans jryans commented Jan 17, 2020

Once cross-signing is the default, we should remove the keyshare dialog.

@turt2live
Copy link
Member

@turt2live turt2live commented Feb 1, 2020

The original change here feels a bit misguided. We still require keyshare requests for RiotX->Web to get information about devices and such. Cross signing does help, though there are legitimate use cases where key sharing is required.

We've backed the original change out for the demo.

@jryans
Copy link
Member Author

@jryans jryans commented Feb 2, 2020

The key share dialog (disabled with cross-signing in matrix-org/matrix-react-sdk#3860) is only used for unverified devices. For verified devices, we send immediately and that has not changed.

It would be good to have more details on this scenario to track down what failed and what should be happening.

@turt2live
Copy link
Member

@turt2live turt2live commented Feb 2, 2020

we were pretty tired at the time - would be worth talking it over with @BillCarsonFr to see what was going on.

@jryans jryans added phase:3 story:1 and removed story:34 labels Feb 11, 2020
@jryans jryans added phase:4 and removed phase:3 labels Mar 16, 2020
@jryans
Copy link
Member Author

@jryans jryans commented Mar 16, 2020

For the record here, there was a bug that prevented auto key sharing with cross-signed device. Fixed by matrix-org/matrix-js-sdk#1243.

This dialog is still planned for removal when cross-signing is out of labs.

@dbkr dbkr self-assigned this May 26, 2020
@dbkr dbkr added this to In Progress in Workflow May 26, 2020
dbkr added a commit to matrix-org/matrix-react-sdk that referenced this issue May 26, 2020
@dbkr dbkr moved this from In Progress to In Review in Workflow May 26, 2020
Workflow automation moved this from In Review to In Test May 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants
You can’t perform that action at this time.