Page MenuHomePhabricator

[native] update session creation code to handle versioning
ClosedPublic

Authored by kamil on Apr 8 2024, 7:46 AM.
Tags
None
Referenced Files
F3394832: D11591.id38919.diff
Sat, Nov 30, 10:38 PM
Unknown Object (File)
Thu, Nov 28, 3:42 AM
Unknown Object (File)
Thu, Nov 28, 3:29 AM
Unknown Object (File)
Thu, Nov 28, 1:48 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
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
Branch
publish-session-resetting
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Apr 9 2024, 6:02 AM
kamil added inline comments.
native/cpp/CommonCpp/CryptoTools/CryptoModule.cpp
265

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

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