Page MenuHomePhabricator

[web] update session creation code to handle versioning
ClosedPublic

Authored by kamil on Mon, Apr 8, 7:25 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 21, 3:40 AM
Unknown Object (File)
Wed, Apr 17, 3:26 AM
Unknown Object (File)
Wed, Apr 17, 12:22 AM
Unknown Object (File)
Mon, Apr 15, 11:45 PM
Unknown Object (File)
Sun, Apr 14, 4:31 PM
Unknown Object (File)
Sat, Apr 13, 4:44 PM
Unknown Object (File)
Sat, Apr 13, 3:14 PM
Unknown Object (File)
Sat, Apr 13, 1:01 PM
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.Tue, Apr 9, 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.Wed, Apr 10, 8:27 AM