Skip to content

A54: Restrict Control-Plane Status Codes#314

Merged
ejona86 merged 8 commits intogrpc:masterfrom
ejona86:restrict-control-plane-status
Aug 24, 2022
Merged

A54: Restrict Control-Plane Status Codes#314
ejona86 merged 8 commits intogrpc:masterfrom
ejona86:restrict-control-plane-status

Conversation

@ejona86
Copy link
Copy Markdown
Member

@ejona86 ejona86 commented Jul 22, 2022

No description provided.

@ejona86 ejona86 requested review from dfawley and markdroth July 22, 2022 19:56
Comment thread A54-restrict-control-plane-status-codes.md
Copy link
Copy Markdown
Member

@markdroth markdroth left a comment

Choose a reason for hiding this comment

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

Looks very good overall! Comments are mostly wording details.

Comment thread A54-restrict-control-plane-status-codes.md Outdated
Comment thread A54-restrict-control-plane-status-codes.md
Comment thread A54-restrict-control-plane-status-codes.md Outdated
Comment thread A54-restrict-control-plane-status-codes.md Outdated
Comment thread A54-restrict-control-plane-status-codes.md Outdated
Comment thread A54-restrict-control-plane-status-codes.md Outdated
Comment thread A54-restrict-control-plane-status-codes.md Outdated
Comment thread A54-restrict-control-plane-status-codes.md Outdated
Comment thread A54-restrict-control-plane-status-codes.md Outdated
Comment thread A54-restrict-control-plane-status-codes.md Outdated
Comment thread A54-restrict-control-plane-status-codes.md
@ejona86 ejona86 requested review from dfawley and markdroth August 22, 2022 22:17
@ejona86 ejona86 merged commit 5ba3845 into grpc:master Aug 24, 2022
@ejona86 ejona86 deleted the restrict-control-plane-status branch August 24, 2022 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants