Page MenuHomePhabricator
Differential D7480

[Tunnelbroker] Create session creation and key messages
ClosedPublic
Actions

Authored by • jon on Apr 17 2023, 6:35 PM.
Tags
None
Referenced Files
F5928640: D7480.id25250.diff
Fri, Apr 18, 2:06 AM
F5923855: D7480.id.diff
Thu, Apr 17, 11:36 PM
F5890817: D7480.diff
Thu, Apr 17, 6:37 AM
F5890506: D7480.id25266.diff
Thu, Apr 17, 6:11 AM
Unknown Object (File)
Thu, Apr 17, 3:43 AM
Unknown Object (File)
Sat, Apr 12, 3:35 AM
Unknown Object (File)
Wed, Apr 9, 3:20 PM
Unknown Object (File)
Tue, Apr 8, 12:44 PM
View All Files
Subscribers
atul
tomek

Details

Reviewers
varun
bartek
• max
ashoat
Commits
rCOMMbb6772194b9a: [Tunnelbroker] Create session creation and key messages
Summary

Remove "legacy" tunnelbroker api messages, and replace with
websocket messages.

Part of https://linear.app/comm/issue/ENG-3676

Depends on D7431

Test Plan

This is more intended to be a design diff. However, deserialization is tested

cd shared/tunnelbroker_messages
cargo test

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• jon created this revision.Apr 17 2023, 6:35 PM
Herald added subscribers: atul, tomek. · View Herald TranscriptApr 17 2023, 6:35 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Apr 17 2023, 6:38 PM
Harbormaster failed remote builds in B18432: Diff 25247!
• jon updated this revision to Diff 25250.Apr 17 2023, 6:49 PM

Rebase on changes

Harbormaster returned this revision to the author for changes because remote builds failed.Apr 17 2023, 6:54 PM
Harbormaster failed remote builds in B18435: Diff 25250!
• jon updated this revision to Diff 25266.Apr 18 2023, 7:59 AM

Rebase on fixes

Harbormaster completed remote builds in B18449: Diff 25266.Apr 18 2023, 8:16 AM
• jon requested review of this revision.Apr 18 2023, 8:16 AM
ashoat resigned from this revision.Apr 18 2023, 9:20 AM

New libraries look good!! Resigning so that somebody else on the team looks at the Rust code

bartek accepted this revision.Apr 18 2023, 11:46 AM

Rust looks good

This revision is now accepted and ready to land.Apr 18 2023, 11:46 AM
Closed by commit rCOMMbb6772194b9a: [Tunnelbroker] Create session creation and key messages (authored by • jon). · Explain WhyApr 19 2023, 10:25 AM
This revision was automatically updated to reflect the committed changes.
• jon added a commit: rCOMMbb6772194b9a: [Tunnelbroker] Create session creation and key messages.
kamil mentioned this in D8752: [Keyserver] Upload new onetime keys to identity service when requested.Sep 13 2023, 6:10 AM

Revision Contents
Changeset List

PathSize
services/
tunnelbroker/
src/
websockets/
2 lines
shared/
tunnelbroker_messages/
1 line
10 lines
src/
3 lines
messages/
28 lines
6 lines
68 lines

Diff 25364

View Options

services/tunnelbroker/src/websockets/messages.rs

Loading...
View Options

services/tunnelbroker/src/websockets/mod.rs

Loading...
View Options

shared/tunnelbroker_messages/.gitignore

Loading...
View Options

shared/tunnelbroker_messages/Cargo.lock

Loading...
View Options

shared/tunnelbroker_messages/Cargo.toml

Loading...
View Options

shared/tunnelbroker_messages/src/lib.rs

Loading...
View Options

shared/tunnelbroker_messages/src/messages/keys.rs

Loading...
View Options

shared/tunnelbroker_messages/src/messages/mod.rs

Loading...
View Options

shared/tunnelbroker_messages/src/messages/session.rs

Loading...