Page MenuHomePhabricator
Differential D9572

[Tunnelbroker] add `MessageID` field to `MessageToDevice`
ClosedPublic
Actions

Authored by kamil on Oct 24 2023, 9:49 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 2, 1:06 PM
Unknown Object (File)
Wed, Oct 2, 1:06 PM
Unknown Object (File)
Wed, Oct 2, 6:01 AM
Unknown Object (File)
Mon, Sep 23, 11:39 AM
Unknown Object (File)
Sat, Sep 7, 1:50 AM
Unknown Object (File)
Aug 24 2024, 3:38 PM
Unknown Object (File)
Jul 31 2024, 5:52 AM
Unknown Object (File)
Jul 31 2024, 5:52 AM
View All 49 Files
Subscribers
ashoat
tomek
will

Details

Reviewers
michal
bartek
varun
• jon
Commits
rCOMMe9eeead4b472: [Tunnelbroker] add `MessageID` field to `MessageToDevice`
Summary

Clients need to know messageID to confirm message receive.
Keeping old in commtest struct to make tests more readable.

Depends on D9571

Test Plan

Run all Tunnelbroker tests (But overall this doesn't affect logic).

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
services/
commtest/
src/
tunnelbroker/
13 lines
tests/
8 lines
8 lines
shared/
tunnelbroker_messages/
src/
messages/
6 lines

Diff 32792

View Options

services/commtest/src/tunnelbroker/socket.rs

Loading...
View Options

services/commtest/tests/tunnelbroker_integration_tests.rs

Loading...
View Options

services/commtest/tests/tunnelbroker_persist_tests.rs

Loading...
View Options

shared/tunnelbroker_messages/src/messages/message_to_device.rs

Loading...