Page MenuHomePhabricator
Differential D8891

[Identity] Adopt TLS-enabled tunnelbroker client
ClosedPublic
Actions

Authored by • jon on Aug 21 2023, 6:59 AM.
Tags
None
Referenced Files
F3362222: D8891.id30269.diff
Sun, Nov 24, 9:49 PM
F3362106: D8891.id30343.diff
Sun, Nov 24, 9:27 PM
F3361807: D8891.diff
Sun, Nov 24, 7:30 PM
Unknown Object (File)
Sat, Nov 23, 1:47 AM
Unknown Object (File)
Mon, Nov 18, 10:26 AM
Unknown Object (File)
Sat, Nov 2, 7:51 AM
Unknown Object (File)
Sun, Oct 27, 2:54 AM
Unknown Object (File)
Sun, Oct 27, 2:54 AM
View All Files
Subscribers
ashoat
tomek

Details

Reviewers
bartek
varun
Commits
rCOMM9563d223292e: [Identity] Adopt TLS-enabled tunnelbroker client
Summary

Use TLS-enabled tunnelbroker client from identity service.

https://linear.app/comm/issue/ENG-4641

Depends on D8890

Test Plan
  • Start identity service, tunnelbroker
cd services/commtest
cargo test --test identity_tunnelbroker_tests

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

• jon created this revision.Aug 21 2023, 6:59 AM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 21 2023, 6:59 AM
Harbormaster completed remote builds in B21947: Diff 30144.Aug 21 2023, 7:17 AM
• jon requested review of this revision.Aug 21 2023, 7:17 AM
bartek accepted this revision.Aug 22 2023, 12:30 AM
This revision is now accepted and ready to land.Aug 22 2023, 12:30 AM
• jon added a child revision: D8916: [Identity] Improve VerifyUserAccessToken logging.Aug 22 2023, 10:41 AM
• jon added a child revision: D8917: [shared/grpc_clients] Export tonic to avoid version mismatch.
• jon updated this revision to Diff 30269.Aug 23 2023, 9:31 AM

Add missed changes

Harbormaster completed remote builds in B22049: Diff 30269.Aug 23 2023, 9:49 AM
• jon updated this revision to Diff 30343.Aug 25 2023, 4:53 PM

Rebase on top of master

Harbormaster completed remote builds in B22109: Diff 30343.Aug 25 2023, 5:12 PM
Closed by commit rCOMM9563d223292e: [Identity] Adopt TLS-enabled tunnelbroker client (authored by • jon). · Explain WhyAug 27 2023, 11:59 PM
This revision was automatically updated to reflect the committed changes.
• jon added a commit: rCOMM9563d223292e: [Identity] Adopt TLS-enabled tunnelbroker client.

Revision Contents
Changeset List

PathSize
services/
identity/
1 line
7 lines
src/
8 lines

Diff 30144

View Options

services/identity/Cargo.lock

Loading...
View Options

services/identity/Cargo.toml

Loading...
View Options

services/identity/build.rs

Loading...
View Options

services/identity/src/tunnelbroker.rs

Loading...