Page MenuHomePhabricator

[native] update session creation code to handle versioning
ClosedPublic

Authored by kamil on Mon, Apr 8, 7:46 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 22, 2:28 AM
Unknown Object (File)
Sun, Apr 21, 1:42 AM
Unknown Object (File)
Tue, Apr 16, 3:28 PM
Unknown Object (File)
Mon, Apr 15, 11:37 PM
Unknown Object (File)
Mon, Apr 15, 2:36 PM
Unknown Object (File)
Mon, Apr 15, 1:58 PM
Unknown Object (File)
Sat, Apr 13, 8:07 PM
Unknown Object (File)
Sat, Apr 13, 8:34 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).

Depends on D11586

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:02 AM
kamil added inline comments.
native/cpp/CommonCpp/CryptoTools/CryptoModule.cpp
265 ↗(On Diff #38919)

handling this param is changed in D11595 - in this diff can be ignored

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