envoyproxy/envoy

Allow filtering of headers sent in CheckRequest to gRPC ext authz

Open

#21 535 ouverte le 1 juin 2022

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/ext_authzarea/grpcenhancementhelp 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: Allow filtering of headers sent in CheckRequest to gRPC ext authz

Description:

The HTTP ext authz allows for the filtering of headers from the request sent to the ext authz service (allowed_headers: https://www.envoyproxy.io/docs/envoy/latest/api-v3/extensions/filters/http/ext_authz/v3/ext_authz.proto#extensions-filters-http-ext-authz-v3-authorizationrequest), but the gRPC version does not.

In my case a lot of non-required headers are being passed along to my ext authz, I would prefer if they were not. I could of course move to straight HTTP but I don't really want to do that either.

Guide contributeur

Direction de recherche
Le problème demande l'ajout d'un filtrage d'en têtes au filtre gRPC ext authz, similaire à la configuration allowed headers de la version HTTP. L'implémentation nécessite de modifier le proto de configuration du filtre gRPC ext authz et la logique de construction de CheckRequest dans le code source C++. Les fichiers pertinents incluent source/extensions/filters/http/ext authz/ et les définitions proto. Comme il n'y a pas de pull requests liées et seulement deux commentaires, ce problème est ouvert à la contribution. La première étape est d'étudier l'implémentation existante d'HTTP ext authz pour comprendre le modèle de filtrage des en têtes, puis de l'adapter pour le chemin gRPC.
Stack technique
cpp
Domaine
backendsecurity
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.
Claire
Prérequis
C++Envoy development environment
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40