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
Unknown Object (File)
Tue, May 7, 1:49 AM
Unknown Object (File)
Sat, May 4, 10:28 PM
Unknown Object (File)
Sat, May 4, 10:30 AM
Unknown Object (File)
Fri, May 3, 7:41 PM
Unknown Object (File)
Thu, May 2, 8:47 PM
Unknown Object (File)
Tue, Apr 30, 1:46 AM
Unknown Object (File)
Sun, Apr 28, 12:16 AM
Unknown Object (File)
Thu, Apr 25, 7:16 PM
View All 79 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/
14 lines

Diff 30343

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...