Page MenuHomePhabricator
Differential D12523

[Tunnelbroker] handle `SetDeviceToken` message
ClosedPublic
Actions

Authored by kamil on Jun 20 2024, 2:54 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:55 PM
Unknown Object (File)
Mon, Dec 23, 2:55 PM
Unknown Object (File)
Sat, Dec 7, 9:47 PM
Unknown Object (File)
Fri, Nov 29, 6:27 AM
Unknown Object (File)
Wed, Nov 27, 8:50 AM
View All 66 Files
Subscribers
ashoat
tomek

Details

Reviewers
bartek
marcin
Commits
rCOMM36a8f028c429: [Tunnelbroker] handle `SetDeviceToken` message
Summary

ENG-8347

Code to handle message from the client.

Depends on D12520, D12522

Test Plan

Tested in next diff

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Jun 20 2024, 2:54 PM
kamil held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptJun 20 2024, 2:54 PM
kamil added a child revision: D12524: [lib] implement sending message to Tunnelbroker.Jun 20 2024, 2:55 PM
Harbormaster failed remote builds in B29845: Diff 41580!Jun 20 2024, 3:00 PM
kamil published this revision for review.Jun 20 2024, 4:12 PM
Harbormaster completed remote builds in B29845: Diff 41580.Jun 20 2024, 4:28 PM
bartek accepted this revision.Jun 21 2024, 11:11 PM
bartek added inline comments.
services/tunnelbroker/src/websockets/session.rs
327–329 ↗(On Diff #41580)

I guess the type can be inferred from handle_message_to_tunnelbroker() call below

338–341 ↗(On Diff #41580)

Wondering why we're using Option::from() everywhere. Simple Some() would work too

This revision is now accepted and ready to land.Jun 21 2024, 11:11 PM
kamil updated this revision to Diff 41801.Jun 28 2024, 6:17 AM

address nits

Harbormaster completed remote builds in B29997: Diff 41801.Jun 28 2024, 6:34 AM
Closed by commit rCOMM36a8f028c429: [Tunnelbroker] handle `SetDeviceToken` message (authored by kamil). · Explain WhyJun 28 2024, 6:48 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM36a8f028c429: [Tunnelbroker] handle `SetDeviceToken` message.

Revision Contents
Changeset List

PathSize
services/
tunnelbroker/
src/
websockets/
55 lines

Diff 41804

View Options

services/tunnelbroker/src/websockets/session.rs

Loading...