Skip to content

Proto validation of gRPC runtime credentials #8010

@htuch

Description

@htuch

In some places, e.g.

// TODO(htuch): This should probably follow the standard metadata validation pattern, but
,
credential.from_plugin(), ProtobufMessage::getNullValidationVisitor(),
, we're explicitly plumbing in a null proto validation visitor, rather than using something derived from server settings. We also, more generally, lack handling of proto validation failure.

In some sense, this is totally fine today, since these are not control plane API and don't use PGV. In principle, it might be useful to have unknown field validation here and PGV support, so opening this issue to track future work here.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementFeature requests. Not bugs or questions.help wantedNeeds help!

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions