[lib] implement sending messages to Tunnelbroker
ClosedPublic
Actions

Authored by kamil on Oct 27 2023, 11:37 AM.

Details

Reviewers

michal
bartek
tomek

Commits

rCOMM9bc16ce3b33c: [lib] implement sending messages to Tunnelbroker

Summary

Code for sending Message to another device via Tunnelbroker.

Depends on D9611

Test Plan

Test if messages are properly sent and promises are resolved/rejected.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Oct 27 2023, 11:37 AM

kamil held this revision as a draft.

Herald added subscribers: will, ashoat. · View Herald TranscriptOct 27 2023, 11:37 AM

kamil added a child revision: D9617: [web] implement screen for testing Tunnelbroker.Oct 27 2023, 11:41 AM

Harbormaster completed remote builds in B23530: Diff 32455.Oct 27 2023, 12:04 PM

kamil published this revision for review.Oct 27 2023, 12:41 PM

kamil added inline comments.

lib/tunnelbroker/tunnelbroker-context.js
112 ↗	(On Diff #32455)	I'll remove this comment before landing, creating a task: ENG-5601.

kamil mentioned this in D9606: [lib] introduce Tunnelbroker Context.Oct 31 2023, 4:44 PM

michal accepted this revision.Nov 2 2023, 2:21 PM

michal added inline comments.

lib/tunnelbroker/tunnelbroker-context.js
114 ↗	(On Diff #32455)	Probably testing remains?

This revision is now accepted and ready to land.Nov 2 2023, 2:21 PM

remove comment
remove debug

lib/tunnelbroker/tunnelbroker-context.js
114 ↗	(On Diff #32455)	yes, thanks

Harbormaster failed remote builds in B23767: Diff 32729!Nov 6 2023, 9:48 AM

rebase

Harbormaster failed remote builds in B23771: Diff 32733!Nov 6 2023, 11:08 AM

Harbormaster completed remote builds in B23771: Diff 32733.Nov 6 2023, 12:18 PM

Closed by commit rCOMM9bc16ce3b33c: [lib] implement sending messages to Tunnelbroker. · Explain WhyNov 6 2023, 4:30 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM9bc16ce3b33c: [lib] implement sending messages to Tunnelbroker.

Revision Contents
Changeset List

Path

Size

lib/

tunnelbroker/

tunnelbroker-context.js

54 lines

Diff 32807

View Options

lib/tunnelbroker/tunnelbroker-context.js

[lib] implement sending messages to TunnelbrokerClosedPublicActions