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

Get type checking from OlmMachine.shareRoomKey #338

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

AndrewFerr
Copy link
Contributor

Update the rust-sdk bindings to have access to type checking in the return value of OlmMachine.shareRoomKey, which now returns an array of ToDeviceRequest objects instead of a JSON encoding of the whole array.

Signed-off-by: Andrew Ferrazzutti [email protected]

Checklist

  • Tests written for all new code
  • Linter has been satisfied
  • Sign-off given on the changes (see CONTRIBUTING.md)

on account of it being removed from the crypto-sdk
OlmMachine.receiveSyncChanges returns an array of [device messages,
room key changes], so emit "to_device.decrypted" with that instead of
the entire array.

Fixes regression introduced by turt2live#287.
Also condense the validity checks on the returned value
Update the rust-sdk bindings to have access to type checking in the
return value of OlmMachine.shareRoomKey, which now returns an array of
ToDeviceRequest objects instead of a JSON encoding of the whole array.
@AndrewFerr
Copy link
Contributor Author

Note that this applies on top of #332 due to incrementing the rust-sdk version from that PR.

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

Successfully merging this pull request may close these issues.

1 participant