envoyproxy/envoy

Pass gRPC metadata to ext_authz

Open

#7 272 ouverte le 14 juin 2019

Voir sur GitHub
 (2 commentaires) (11 réactions) (0 assignés)C++ (5 373 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

Title: Pass gRPC metadata to ext_authz

Description: With current ext_authz filter implementation an external grpc_service receives only subset of request information, such as IPs and TLS details. It would be really helpful to have ability to access gRPC metadata headers as well.

Our current authorization schema is based on metadata fields added to grpc calls. Metadata is checked on server side, and we would like to move it out to Envoy filters.

It seems to be possible for http_service to configure which http headers will be passed to authz service, but nothing like that is configurable for grpc_service type.

Guide contributeur

Direction de recherche
Le problème demande d'ajouter la capacité de passer des métadonnées gRPC au filtre ext authz pour les services gRPC. Actuellement, seuls les en têtes HTTP peuvent être configurés. L'implémentation doit permettre de configurer quelles clés de métadonnées gRPC transférer, similaire à la configuration allowed headers pour HTTP. Les principaux fichiers à modifier se trouvent dans l'implémentation du filtre ext authz, probablement sous source/extensions/filters/http/ext authz/ et le client gRPC. La discussion dans le problème et les PR liés (aucun) peut fournir le contexte.
Stack technique
cppgrpc
Domaine
backendsecurityapi
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
20