Page MenuHomePhabricator
Differential D13046

[lib] reset session and resend message when device is unable to decrypt messages
ClosedPublic
Actions

Authored by kamil on Aug 12 2024, 2:49 AM.
Tags
None
Referenced Files
F2715454: D13046.id43304.diff
Mon, Sep 16, 3:17 AM
Unknown Object (File)
Sun, Sep 15, 9:29 AM
Unknown Object (File)
Sun, Sep 15, 9:06 AM
Unknown Object (File)
Sun, Sep 15, 9:06 AM
Unknown Object (File)
Wed, Sep 11, 4:31 AM
Unknown Object (File)
Tue, Sep 10, 5:19 PM
Unknown Object (File)
Fri, Sep 6, 5:27 PM
Unknown Object (File)
Fri, Sep 6, 5:26 PM
View All 34 Files
Subscribers
ashoat

Details

Reviewers
tomek
marcin
Commits
rCOMM575677018ac0: [lib] reset session and resend message when device is unable to decrypt messages
Summary

ENG-6982.

We want to reset session when there was an error while decrypting.

Depends on D13033

Test Plan
  1. Create a session between devices A and B
  2. On B comment out sending confirmation.
  3. Send some messages from A to B
  4. On B simulate Olm error.
  5. Test that all messages are sent again and decrypted successfully.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Aug 12 2024, 2:49 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptAug 12 2024, 2:49 AM
kamil published this revision for review.Aug 12 2024, 3:04 AM
kamil added inline comments.
lib/utils/olm-utils.js
126–129 ↗(On Diff #43304)

We no longer track order and not using sequential decrypt

Harbormaster failed remote builds in B31040: Diff 43304!Aug 12 2024, 3:07 AM
kamil added a child revision: D13047: [SQLite] recreate table for Inbound P2P messages.Aug 12 2024, 6:43 AM
tomek accepted this revision.Mon, Aug 19, 5:46 AM
tomek added inline comments.
lib/utils/olm-utils.js
123 ↗(On Diff #43304)
This revision is now accepted and ready to land.Mon, Aug 19, 5:46 AM
kamil updated this revision to Diff 43498.Tue, Aug 20, 3:53 AM

address review

Harbormaster completed remote builds in B31168: Diff 43498.Tue, Aug 20, 4:10 AM
Closed by commit rCOMM575677018ac0: [lib] reset session and resend message when device is unable to decrypt messages (authored by kamil). · Explain WhyWed, Aug 21, 2:57 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM575677018ac0: [lib] reset session and resend message when device is unable to decrypt messages.

Revision Contents
Changeset List

PathSize
lib/
tunnelbroker/
36 lines
utils/
9 lines

Diff 43544

View Options

lib/tunnelbroker/use-peer-to-peer-message-handler.js

Loading...
View Options

lib/utils/olm-utils.js

Loading...