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

Invisible crypto: revisit CryptoMode concept #4428

Closed
richvdh opened this issue Sep 23, 2024 · 0 comments · Fixed by #4429
Closed

Invisible crypto: revisit CryptoMode concept #4428

richvdh opened this issue Sep 23, 2024 · 0 comments · Fixed by #4429

Comments

@richvdh
Copy link
Member

richvdh commented Sep 23, 2024

#4407 introduced a CryptoMode api to the js-sdk, which we think we need to revisit, for a couple of reasons:

  1. "CryptoMode" isn't very descriptive (does it refer to the crypto ciphers being used? MLS vs Double Ratchet? or?)
  2. More importantly, the exposed options don't actually fit what we need very well.

Instead, we imagine we might have a CryptoApi.SetDeviceIsolationMode, which takes some sort of enum-with-options parameter.

(For Element folks, see brainstorming doc at https://docs.google.com/spreadsheets/d/1OEoqP3PAEsdvPBCYxIxJdp0GZhtLbq75dex-fOVQkO0/edit?usp=sharing)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants