fix broken v1alpha gRPC reflection support #1718
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
fixes gRPC reflection support for clients like Postman
with the introduction of v1 Reflection,
grpcutil
was updated in a way that brokev1alpha
reflection calls.grpc-go
continues to support both versions and the handler forv1alpha
was being registered, but the authless wrapper exposed bygrpcutil
was failing to bypass auth forv1alpha
calls because it dropped support for it.This vendors the changes to
grpcutil
that introduce backv1alpha
support, and also addresses a missing allowlist in the consistency middleware for thev1
version.