Page MenuHomePhabricator
Differential D5838

[services/commtest] Tunnelbroker - Adding of helper function for the `NewSession` client request
ClosedPublic
Actions

Authored by • max on Dec 7 2022, 5:45 AM.
Tags
None
Referenced Files
F3495864: D5838.id19552.diff
Thu, Dec 19, 10:21 AM
F3495858: D5838.id19208.diff
Thu, Dec 19, 10:18 AM
F3495850: D5838.id20181.diff
Thu, Dec 19, 10:14 AM
F3495815: D5838.id20176.diff
Thu, Dec 19, 10:07 AM
F3495734: D5838.id20180.diff
Thu, Dec 19, 9:48 AM
F3495413: D5838.id20172.diff
Thu, Dec 19, 8:33 AM
F3494889: D5838.diff
Thu, Dec 19, 7:04 AM
F3491111: D5838.id20181.diff
Wed, Dec 18, 6:17 PM
View All Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
• jon
marcin
Commits
rCOMM73fe7a8927a1: [services/commtest] Tunnelbroker - Adding of helper function for the…
Summary

This diff introduces adding the helper method to the Commtest integration API tests with the gRPC client to call the NewSession request to the Tunnelbroker server.
This function is called in D5931.

Linear task: ENG-1657

Test Plan

The commtest app is successfully built and the tests are successfully passed in the following D5931.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• max created this revision.Dec 7 2022, 5:45 AM
• max held this revision as a draft.
Herald added a reviewer: • jon. · View Herald TranscriptDec 7 2022, 5:45 AM
Herald added subscribers: atul, tomek, ashoat. · View Herald Transcript
• max added a child revision: D5837: [services/commtest] Tunnelbroker - Adding of helper function for the `SessionSignature` client request.Dec 7 2022, 5:45 AM
• max edited the summary of this revision. (Show Details)Dec 7 2022, 5:53 AM
• max added a reviewer: bartek.
• max edited the summary of this revision. (Show Details)
• max removed a child revision: D5837: [services/commtest] Tunnelbroker - Adding of helper function for the `SessionSignature` client request.
• max added a parent revision: D5837: [services/commtest] Tunnelbroker - Adding of helper function for the `SessionSignature` client request.
Harbormaster completed remote builds in B14033: Diff 19208.Dec 7 2022, 5:57 AM
• max updated this revision to Diff 19552.Dec 19 2022, 6:37 AM

Rebasing on the stack changes.

• max edited parent revisions, added: D5927: [services/commtest] Tunnelbroker - Adding of helper function for the string signing with the private key; removed: D5837: [services/commtest] Tunnelbroker - Adding of helper function for the `SessionSignature` client request.Dec 19 2022, 6:38 AM
• max added a child revision: D5928: [services/commtest] Tunnelbroker - Adding the tonic client builder function.Dec 19 2022, 6:43 AM
Harbormaster completed remote builds in B14310: Diff 19552.Dec 19 2022, 6:48 AM
• max published this revision for review.Dec 19 2022, 7:08 AM
• max edited the summary of this revision. (Show Details)
• max edited the test plan for this revision. (Show Details)
• max edited reviewers, added: marcin; removed: bartek. • max added 1 blocking reviewer(s): • jon.
• jon accepted this revision.Dec 20 2022, 9:52 AM
This revision is now accepted and ready to land.Dec 20 2022, 9:52 AM
• max updated this revision to Diff 20172.Dec 27 2022, 6:04 AM
• max edited the test plan for this revision. (Show Details)

Rebasing on master.

Harbormaster completed remote builds in B14791: Diff 20172.Dec 27 2022, 6:18 AM
• max updated this revision to Diff 20176.Dec 27 2022, 6:22 AM

Rebasing and fix merging.

Harbormaster completed remote builds in B14793: Diff 20176.Dec 27 2022, 6:36 AM
• max updated this revision to Diff 20180.Dec 27 2022, 6:43 AM

Removing of the dead commit to fix merging.

Harbormaster completed remote builds in B14797: Diff 20180.Dec 27 2022, 6:56 AM
Closed by commit rCOMM73fe7a8927a1: [services/commtest] Tunnelbroker - Adding of helper function for the… (authored by • max). · Explain WhyDec 27 2022, 6:57 AM
This revision was automatically updated to reflect the committed changes.
• max added a commit: rCOMM73fe7a8927a1: [services/commtest] Tunnelbroker - Adding of helper function for the….

Revision Contents
Changeset List

PathSize
services/
commtest/
src/
tunnelbroker/
29 lines

Diff 20181

View Options

services/commtest/src/tunnelbroker/new_session.rs

Loading...