Page MenuHomePhabricator

[web][keyserver] Fork Olm
ClosedPublic

Authored by ashoat on Mar 16 2023, 6:33 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 25, 2:12 PM
Unknown Object (File)
Sat, May 25, 2:12 PM
Unknown Object (File)
Sat, May 25, 2:12 PM
Unknown Object (File)
Sat, May 25, 2:12 PM
Unknown Object (File)
Sat, May 25, 2:12 PM
Unknown Object (File)
Sat, May 25, 2:11 PM
Unknown Object (File)
Fri, May 24, 2:42 PM
Unknown Object (File)
Tue, May 21, 12:40 PM
Subscribers

Details

Summary

This switches us over to using a fork of Olm latest.

The fork is mostly Anunay's work to add X3DH support, plus @jon's work to add Nix build support for macOS, plus some fixes from me.

To review this you'll probably want to check the fork's commit history on GitHub. We should probably set up some way to review the stuff in the fork, but I'm not sure what the best approach is yet. For now, I've left comments directly on @anunay's commit on GitHub.

Test Plan
  1. Unit tests are added in D7090
  2. Also note that we successfully ran Olm C++ and JS test suites
  3. I also patched the web app to confirm it worked

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Mar 16 2023, 6:44 PM
Harbormaster failed remote builds in B17376: Diff 23809!

RE CI failure, looks like we need to update the xcode folder in the fork as well. Will publish this for review for now, but will definitely make sure to fix the xcode folder before landing

ashoat retitled this revision from Fork Olm to [web][keyserver] Fork Olm.Mar 16 2023, 7:33 PM

Pulled out native side to D7091 since it turned out to be more work than expected

Update to 0.0.2... it doesn't actually affect web or keyserver, but I figure it's better to do it

This revision is now accepted and ready to land.Mar 17 2023, 12:48 PM
This revision was automatically updated to reflect the committed changes.