Page MenuHomePhabricator

[Tunnelbroker][lib] cleanup Device <-> Tunnelbroker message types
ClosedPublic

Authored by kamil on Jul 23 2024, 5:53 AM.
Tags
None
Referenced Files
F3377326: D12856.diff
Wed, Nov 27, 5:11 AM
Unknown Object (File)
Sat, Nov 23, 2:47 PM
Unknown Object (File)
Sat, Nov 9, 8:48 AM
Unknown Object (File)
Fri, Nov 1, 11:51 AM
Unknown Object (File)
Oct 18 2024, 10:07 AM
Unknown Object (File)
Oct 18 2024, 3:21 AM
Unknown Object (File)
Oct 14 2024, 1:02 PM
Unknown Object (File)
Oct 14 2024, 1:02 PM
Subscribers

Details

Summary

ENG-8621.

This improves readability, we can distingiush direction of message sender and recipient.

Test Plan

This only changes types, should not affect logic or content of messages sent between device <-> Tunnelbroker so only yarn flow:all and cargo check.

Diff Detail

Repository
rCOMM Comm
Branch
tb
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Jul 23 2024, 6:20 AM
tomek added inline comments.
lib/tunnelbroker/tunnelbroker-context.js
300

Are you sure this is the right type? Shouldn't be deviceToTunnelbroker?

This revision is now accepted and ready to land.Jul 24 2024, 6:03 AM
kamil edited the summary of this revision. (Show Details)

use deviceToTunnelbroker