Page MenuHomePhabricator

[lib] implement sending message to Tunnelbroker
ClosedPublic

Authored by kamil on Jun 20 2024, 2:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 23, 2:56 PM
Unknown Object (File)
Mon, Dec 23, 2:56 PM
Unknown Object (File)
Mon, Dec 23, 2:56 PM
Unknown Object (File)
Mon, Dec 23, 2:56 PM
Unknown Object (File)
Mon, Dec 23, 2:55 PM
Unknown Object (File)
Mon, Dec 23, 2:55 PM
Unknown Object (File)
Thu, Dec 12, 5:58 PM
Unknown Object (File)
Fri, Dec 6, 6:52 PM
Subscribers

Details

Summary

ENG-8347

Add callback to send message to Tunnelbroker.

Depends on D12523

Test Plan

Call this from both web and native:

const message: SetDeviceToken = {
  type: messageToTunnelbrokerTypes.SET_DEVICE_TOKEN,
  deviceToken: 'token',
};
 await sendMessageToTunnelbroker(JSON.stringify(message));

and verify if deviceToken is persisted in DDB.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Jun 20 2024, 4:13 PM
lib/tunnelbroker/tunnelbroker-context.js
363–364 ↗(On Diff #41581)

Leftover?

lib/tunnelbroker/tunnelbroker-context.js
363–364 ↗(On Diff #41581)

yeah

This revision is now accepted and ready to land.Jun 24 2024, 11:44 PM
lib/tunnelbroker/tunnelbroker-context.js
437–438 ↗(On Diff #41806)

The naming here is a bit confusing. Should sendMessage be renamed to something that makes it contrast more clearly with sendMessageToTunnelbroker?

lib/tunnelbroker/tunnelbroker-context.js
437–438 ↗(On Diff #41806)

point 1 in ENG-8621