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

alts: Reduce ALTS counter overflow length from 5 to 4. #6699

Merged

Conversation

matthewstevenson88
Copy link
Contributor

@matthewstevenson88 matthewstevenson88 commented Oct 6, 2023

This change is being made for consistency with the changes we're making to our internal implementation.

RELEASE NOTES: none

@matthewstevenson88 matthewstevenson88 added this to the 1.60 Release milestone Oct 6, 2023
@matthewstevenson88 matthewstevenson88 added the Type: Security A bug or other problem affecting security label Oct 6, 2023
@matthewstevenson88 matthewstevenson88 marked this pull request as ready for review October 6, 2023 18:21
@arvindbr8 arvindbr8 requested a review from cesarghali October 16, 2023 22:03
Copy link
Member

@arvindbr8 arvindbr8 left a comment

Choose a reason for hiding this comment

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

LGTM

@arvindbr8 arvindbr8 merged commit 7b8d0fd into grpc:master Oct 24, 2023
11 checks passed
matthewstevenson88 added a commit that referenced this pull request Oct 24, 2023
arvindbr8 pushed a commit to arvindbr8/grpc-go that referenced this pull request Nov 7, 2023
arvindbr8 pushed a commit to arvindbr8/grpc-go that referenced this pull request Nov 7, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Security A bug or other problem affecting security
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants