[lib] add sending Tunnelbroker message with already defined `messageID`
ClosedPublic
Actions

Authored by kamil on May 16 2024, 8:53 AM.

Details

Reviewers

tomek
bartek

Commits

rCOMM1ee2cddd84c4: [lib] add sending Tunnelbroker message with already defined `messageID`

Summary

Messages to other devices are generated when processing store ops, that's when messageID is generated, and we want to send it without generating new one.

Test Plan

Test sending Tunnelbroker messsages

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.May 16 2024, 8:53 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptMay 16 2024, 8:53 AM

Harbormaster completed remote builds in B28989: Diff 40293.May 16 2024, 9:09 AM

kamil added a child revision: D12077: [lib] add `deviceID` prop to Tunnelbroker's `MessageProcessed`.May 16 2024, 9:12 AM

kamil published this revision for review.May 16 2024, 9:39 AM

bartek accepted this revision.May 17 2024, 2:03 AM

This revision is now accepted and ready to land.May 17 2024, 2:03 AM

bartek added inline comments.May 17 2024, 2:05 AM

lib/tunnelbroker/tunnelbroker-context.js
388–392 ↗	(On Diff #40293)	I was thinking about an alternative approach - this might save some from introducing `sendMessageWithID` but it might have more drawbacks that I'm not aware of.