Page MenuHomePhabricator

[lib] update types to allow missing OTK when creating session
ClosedPublic

Authored by kamil on May 23 2024, 4:23 AM.
Tags
None
Referenced Files
F3366795: D12204.diff
Mon, Nov 25, 12:29 PM
Unknown Object (File)
Fri, Nov 8, 1:26 PM
Unknown Object (File)
Oct 19 2024, 2:14 AM
Unknown Object (File)
Oct 18 2024, 12:17 PM
Unknown Object (File)
Oct 17 2024, 3:20 PM
Unknown Object (File)
Oct 17 2024, 10:48 AM
Unknown Object (File)
Oct 14 2024, 4:27 PM
Unknown Object (File)
Oct 14 2024, 4:27 PM
Subscribers

Details

Summary

Modifying types and callsites to reflect new olm API.

Depends on D12203

Test Plan

Tested end-to-end:

  1. Tested content session with OTKs on both web and native
  2. Tested content session without OTKs on both web native
  3. Tested notifs
  4. Tested backward compatibility: @marcin was running his devices logged to staging on old olm version, and I was running the new version (this stack). WIth one OTKs it works as previously (tested both me and Marcin sending requests to each other and tested both platform). When there are no OTKs Marcin was unable to create a session request (same as now), I was able to create a session request, but it failed on Martin’s device with BAD_MESSAGE_KEY_ID but without crashing the app.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.May 23 2024, 5:50 AM

Note to myself to update the code after D12131 is landed

This revision is now accepted and ready to land.May 28 2024, 4:39 AM
  • update createOlmSessionWithPeer
  • rebase before landing