[lib][web][keyserver] replace Olm with Vodozemac's Olm
AbandonedPublic
Actions

Authored by ashoat on Thu, Nov 27, 2:41 PM.

Details

Reviewers

kamil

Summary

ENG-11531.

There is one thing with naming which is not obvious, this is using Olm name - Olm is still part of Vodozemac crate, so I think this is still correct, but I am okay with updating this too.

I tried to make API as similar as it is possible, so there are files when changing import is everything we have to do.

I redefined EncryptResult in lib/. Importing it from Olm was bad idea because it was used on native too, which causes Olm/Vodozemac wasm to be bundled into native, which worked with Olm but not with Vodozemac.

Depends on D15547

Test Plan

Test encrypt/decrypt and creating session with Olm and Vodozemac.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Thu, Nov 27, 2:41 PM

kamil held this revision as a draft.

Herald added a subscriber: tomek. · View Herald TranscriptThu, Nov 27, 2:41 PM

Harbormaster failed remote builds in B36059: Diff 50848!Thu, Nov 27, 2:44 PM

kamil added a child revision: D15549: [native] implement CXX bindings to Vodozemac.Thu, Nov 27, 2:45 PM

kamil published this revision for review.Thu, Nov 27, 2:58 PM

kamil mentioned this in rCOMM1a4fcb98254b: [lib][web][keyserver] add Vodozemac library and `.wasm` file handling (#324).Thu, Dec 4, 3:41 PM

This was landed, but it got squashed into another commit due to the way we handled the corresponding GitHub PRs (Phabricator was down at the time, so we used GitHub)