Page MenuHomePhabricator

[web] update session creation code to handle versioning
ClosedPublic

Authored by kamil on Apr 8 2024, 7:25 AM.
Tags
None
Referenced Files
F3359131: D11586.diff
Sun, Nov 24, 7:27 AM
Unknown Object (File)
Sun, Nov 3, 8:12 AM
Unknown Object (File)
Oct 12 2024, 10:23 PM
Unknown Object (File)
Oct 12 2024, 10:23 PM
Unknown Object (File)
Oct 12 2024, 10:23 PM
Unknown Object (File)
Oct 12 2024, 10:23 PM
Unknown Object (File)
Oct 12 2024, 10:23 PM
Unknown Object (File)
Sep 23 2024, 11:31 AM
Subscribers

Details

Summary

This code:

  1. Updates contentInboundSessionCreator to decide whether to accept new or keep old session depending on sessionVersion
  2. Updates contentOutboundSessionCreator to return the session version (right now by default we override session and bump counter).
  3. Update Tunnelbroker message type to send sessionVersion along with init message

Depends on D11583

Test Plan
  1. Make sure session creation works
  2. Outbound session creation cause bumping counter
  3. Inbound session creation success or throw an appropriate error (tested all cases)

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.Apr 9 2024, 6:01 AM
kamil added inline comments.
native/crypto/olm-api.js
22 ↗(On Diff #38936)

removed in D11591

40 ↗(On Diff #38936)

removed in D11591

This revision is now accepted and ready to land.Apr 10 2024, 8:27 AM