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

Complete rollout of Rust crypto stack #27560

Closed
richvdh opened this issue Jun 12, 2024 · 1 comment · Fixed by matrix-org/matrix-react-sdk#12630
Closed

Complete rollout of Rust crypto stack #27560

richvdh opened this issue Jun 12, 2024 · 1 comment · Fixed by matrix-org/matrix-react-sdk#12630
Assignees
Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust T-Enhancement X-Release-Blocker

Comments

@richvdh
Copy link
Member

richvdh commented Jun 12, 2024

Since matrix-org/matrix-react-sdk#12203 (Element Web 1.11.58), all new logins on all deployments of Element Web/Desktop have been using the Rust crypto stack.

In addition:

We have not seen any particular causes for concern. I therefore think it is time to complete the rollout. In particular:

  • All remaining existing logins in Element Desktop should be migrated to Rust crypto.
  • All remaining existing logins in all deployments of Element Web should be migrated to Rust crypto.

Part of #27001

@dosubot dosubot bot added A-Element-R Issues affecting the port of Element's crypto layer to Rust T-Enhancement labels Jun 12, 2024
@richvdh
Copy link
Member Author

richvdh commented Jun 12, 2024

I think the best way to do this is just to update MatrixClientPeg.initClientCrypto so that, effectively, useRustCrypto is always true, and remove the feature_rust_crypto and RustCrypto.staged_rollout_percent settings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Element-R Issues affecting the port of Element's crypto layer to Rust T-Enhancement X-Release-Blocker
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant