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

Authored by kamil on Fri, Aug 23, 5:13 AM.

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

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Fri, Aug 23, 5:13 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptFri, Aug 23, 5:14 AM

kamil added a child revision: D13151: [lib] deprecate `sendDMOperation` and implement method to create sending promise.Fri, Aug 23, 5:19 AM

Harbormaster completed remote builds in B31234: Diff 43607.Fri, Aug 23, 5:29 AM

kamil published this revision for review.Fri, Aug 23, 7:25 AM

tomek accepted this revision.Mon, Aug 26, 3:22 AM

tomek added inline comments.

lib/tunnelbroker/peer-to-peer-context.js
206 ↗	(On Diff #43607)	We should add a code comment explaining why this function returns a promise that almost always resolves (even when all the messages fail).