Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[vnet] gprc service protos #51215

Merged
merged 2 commits into from
Jan 23, 2025
Merged

[vnet] gprc service protos #51215

merged 2 commits into from
Jan 23, 2025

Conversation

nklaassen
Copy link
Contributor

This PR adds protos for the VNet client application service, to be used for Windows support, based on RFD 195

// Hash specifies a cryptographic hash function.
enum Hash {
HASH_UNSPECIFIED = 0;
HASH_NONE = 1;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When is HASH_NONE used? It might be worth adding a comment about this, I'm not sure (or refer to another place which explains this).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's used for Ed25519 signature which do no pre-hash, added a comment

@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from kopiczko January 23, 2025 18:38
@nklaassen nklaassen enabled auto-merge January 23, 2025 22:16
@nklaassen nklaassen added this pull request to the merge queue Jan 23, 2025
Merged via the queue into master with commit 678c161 Jan 23, 2025
43 checks passed
@nklaassen nklaassen deleted the nklaassen/vnet-protos branch January 23, 2025 22:55
@public-teleport-github-review-bot

@nklaassen See the table below for backport results.

Branch Result
branch/v17 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/branch/v17 no-changelog Indicates that a PR does not require a changelog entry size/md vnet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants