Page MenuHomePhabricator

[lib] create session with peer only when it is missing
ClosedPublic

Authored by kamil on Aug 8 2024, 4:06 AM.
Tags
None
Referenced Files
F2766207: D13027.diff
Thu, Sep 19, 3:16 PM
Unknown Object (File)
Fri, Sep 13, 12:53 PM
Unknown Object (File)
Sat, Sep 7, 1:47 PM
Unknown Object (File)
Sat, Sep 7, 11:37 AM
Unknown Object (File)
Sat, Sep 7, 11:36 AM
Unknown Object (File)
Fri, Sep 6, 5:22 AM
Unknown Object (File)
Sun, Sep 1, 3:00 AM
Unknown Object (File)
Fri, Aug 30, 8:50 PM
Subscribers

Details

Summary

ENG-6982.

ENG-8527.

We definitely shouldn't attempt to create a session when sending a message using Tunnelbroker fails.
According to Olm docs encrypt could throw only two errors:

  • NOT_ENOUGH_RANDOM
  • OUTPUT_BUFFER_TOO_SMALL

and attempting to create a session for those two is not to attempt new session creation, the only case is when the session is missing.

The alternative is to not update the code because peerOlmSessionsCreator is idempotent but we can avoid additional worker/CommCoreModule calls.

Depends on D13025

Test Plan

Test that when session is missing, it is created.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable