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

Olm/Megolm V2 #1557

Open
richvdh opened this issue Jun 7, 2023 · 3 comments
Open

Olm/Megolm V2 #1557

richvdh opened this issue Jun 7, 2023 · 3 comments
Labels
feature Suggestion for a significant extension which needs considerable consideration

Comments

@richvdh
Copy link
Member

richvdh commented Jun 7, 2023

We have some changes that we'd like to make to the Olm and Megolm protocols; starting a list here to keep track of them.

Megolm:

Olm:

  • Extended MAC to improve security
@richvdh
Copy link
Member Author

richvdh commented Jun 7, 2023

Another idea for the list (with credit to @bradtgmurray): use url-safe base64 encoding for megolm session ids to make them easier to use in requests like GET /_matrix/client/v3/room_keys/keys/{roomId}/{sessionId}.

@richvdh richvdh added the feature Suggestion for a significant extension which needs considerable consideration label Jun 8, 2023
@uhoreg
Copy link
Member

uhoreg commented Feb 14, 2024

Megolm:

  • add client's timestamp in ciphertext to make it easier to detect replays
  • message franking?

Olm:

Both:

  • use AES-GCM instead of CBC?

@dkasak
Copy link
Member

dkasak commented Jun 6, 2024

Matrix crypto layer:

  • Dropping the Ed25519 device key in lieu of signing with the Curve25519 device key using XEdDSA.
  • Mandating that the device ID equals the base64-encoding of the Curve25519 device key.

While these are not strictly Olm related in a lower level sense, it would make sense to include them as part of a switch to a hypothetical m.olm.v2.curve25519-aes-sha2 algorithm since that's going to be a breaking change anyway.

use AES-GCM instead of CBC?

We should also consider going further an abandoning AES in favour of (X)ChaCha20-Poly1305.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Suggestion for a significant extension which needs considerable consideration
Projects
None yet
Development

No branches or pull requests

3 participants