Page MenuHomePhabricator
Differential D13150

[lib] update `processOutboundP2PMessages` to return IDs of messages queued on Tunnelbroker
ClosedPublic
Actions

Authored by kamil on Aug 23 2024, 5:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 29, 5:20 AM
Unknown Object (File)
Fri, Nov 29, 5:04 AM
Unknown Object (File)
Mon, Nov 25, 6:22 PM
Unknown Object (File)
Mon, Nov 25, 2:48 PM
Unknown Object (File)
Thu, Nov 21, 7:39 AM
Unknown Object (File)
Sat, Nov 9, 9:02 PM
Unknown Object (File)
Sat, Nov 9, 7:13 PM
Unknown Object (File)
Fri, Nov 8, 11:06 PM
View All 54 Files
Subscribers
ashoat

Details

Reviewers
tomek
inka
Commits
rCOMMd930bcca7a92: [lib] update `processOutboundP2PMessages` to return IDs of messages queued on…
Summary

ENG-8423.

To implement retries, we need to track which P2P messages are sent, which are not.

This code is bad in terms of performance - for each deviceID we can create a Promise and avoid awaiting in sequence, this will be improved as a follow-up.

Depends on D13149

Test Plan

Await promise and check if IDs are returned.

Note: This stack is tested end-to-end according to the process described here.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Aug 23 2024, 5:13 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptAug 23 2024, 5:14 AM
kamil added a child revision: D13151: [lib] deprecate `sendDMOperation` and implement method to create sending promise.Aug 23 2024, 5:19 AM
Harbormaster completed remote builds in B31234: Diff 43607.Aug 23 2024, 5:29 AM
kamil published this revision for review.Aug 23 2024, 7:25 AM
tomek accepted this revision.Aug 26 2024, 3:22 AM
tomek added inline comments.
lib/tunnelbroker/peer-to-peer-context.js
206

We should add a code comment explaining why this function returns a promise that almost always resolves (even when all the messages fail).

This revision is now accepted and ready to land.Aug 26 2024, 3:22 AM
kamil updated this revision to Diff 43633.Aug 26 2024, 6:28 AM

add code comment

Harbormaster completed remote builds in B31258: Diff 43633.Aug 26 2024, 6:45 AM
Closed by commit rCOMMd930bcca7a92: [lib] update `processOutboundP2PMessages` to return IDs of messages queued on… (authored by kamil). · Explain WhyAug 26 2024, 9:05 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMMd930bcca7a92: [lib] update `processOutboundP2PMessages` to return IDs of messages queued on….

Revision Contents
Changeset List

PathSize
lib/
tunnelbroker/
79 lines

Diff 43607

View Options

lib/tunnelbroker/peer-to-peer-context.js

Loading...