Paths

Table of Contentst

Differential D6040

[native/rust_library] Tunnelbroker client - Adding of the `get_new_session_id` function
AcceptedPublic
Actions

Authored by • max on Dec 27 2022, 3:34 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Jun 20, 9:18 PM

	Unknown Object (File)
	Thu, Jun 20, 2:54 AM

	Unknown Object (File)
	Wed, Jun 19, 10:11 AM

	Unknown Object (File)
	Mon, Jun 17, 11:00 AM

	Unknown Object (File)
	Wed, Jun 12, 11:34 AM

	Unknown Object (File)
	Sun, Jun 9, 8:52 PM

	Unknown Object (File)
	Sat, Jun 1, 9:43 PM

	Unknown Object (File)
	Thu, May 30, 3:41 AM

View All 58 Files

Subscribers

Details

Reviewers

• jon
bartek

Summary

This diff introduces the get_new_session_id internal function which is a gRPC unary request to the Tunnelbroker server to create a new session and obtain a SessionID. The function gets the pointer to the Tonic client, and new session parameters (including signed nonce from the D6039, D5999), and returns the sessionID from the server.

Linear task: ENG-2434

Test Plan

Passing CI gates is enough because the Rust analyzer will catch any disparity between the gRPC request method and the gRPC proto schema.

Diff Detail

Repository

Branch

native-tb-client-add-new-session-request

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

• max created this revision.Dec 27 2022, 3:34 AM

• max held this revision as a draft.

Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptDec 27 2022, 3:34 AM

• max added a parent revision: D6039: [native/rust_library] Tunnelbroker client - Adding of the `get_nonce_to_sign` function.Dec 27 2022, 3:35 AM

Harbormaster completed remote builds in B14773: Diff 20151.Dec 27 2022, 3:48 AM

• max published this revision for review.Dec 27 2022, 3:51 AM

• max edited the summary of this revision. (Show Details)

• max edited the test plan for this revision. (Show Details)

• max added reviewers: • jon, bartek.

Rubber stamping as there's no "smells", would like to eventually see a commtest around this

This revision is now accepted and ready to land.Dec 27 2022, 10:39 AM

Rebasing and merging.

Harbormaster failed remote builds in B14885: Diff 20302!Dec 29 2022, 2:23 AM

Revision Contents
Changeset List

Path

Size

native/

native_rust_library/

src/

tunnelbroker_client.rs

33 lines

Diff 20302

native/native_rust_library/src/tunnelbroker_client.rs

Loading...