[web] update `olm` session creation code use `EncryptedData`
ClosedPublic
Actions

Authored by kamil on Mar 28 2024, 10:29 AM.

Details

Reviewers

michal
marcin

Commits

rCOMM8b91ec580024: [web] update `olm` session creation code use `EncryptedData`

Summary

Context in ENG-7228.

Note: We use a shared olmAPI interface on both platforms, to split this logically into diffs sometimes I had to make changes on one platform and make some temporary changes on another platform to make sure CI passes but this approach is consistent and uses the same approach for both web and native.
Depends on D11449

Test Plan

Tested in D11451.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Mar 28 2024, 10:29 AM

kamil held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMar 28 2024, 10:29 AM

kamil added a child revision: D11451: [native] update `olm` session creation code use `EncryptedData`.Mar 28 2024, 10:30 AM

Harbormaster completed remote builds in B27788: Diff 38478.Mar 28 2024, 10:48 AM

kamil edited the test plan for this revision. (Show Details)Mar 29 2024, 5:38 AM

kamil published this revision for review.Mar 29 2024, 5:44 AM

kamil edited the summary of this revision. (Show Details)

kamil added inline comments.

lib/components/keyserver-connection-handler.js
164	We decided to not update the notif's code to use full `EncryptedMessage`, so sending only the encrypted message here to avoid making the endpoint accept inconsistent `initial*EncryptedMessage`