envoyproxy/envoy

Route Matching during filter dispatch

Open

#6824 aperta il 6 mag 2019

Vedi su GitHub
 (6 commenti) (2 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/httpenhancementhelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

At present envoy does not support route matching to selectively dispatch a filter.

Nginx/Apache support configs like the following

<Location "/super-secret">
    AuthBasicProvider ldap
   AuthLDAPURL ldap://127.0.0.1/dc=example,dc=com?uid?one
    Require valid-user
</Location>

This config uses the ldap filter only when a route is matched. At present every filter must support an ad-hoc enable-disable flag. This should be supported directly.

Options:

  • Add RouteMatch to HTTPFilter
  • Standardize on request dynamic metadata key such as filter_name.enabled. This metadata can then be added as part of a route. httpConnectionManager should check this value before dispatching.

Guida contributor

Direzione di ricerca
Esplora la logica di dispatch dei filtri del connection manager HTTP in source/common/http/filter/. Comprendi come i filtri sono attualmente abilitati/disabilitati. Poi considera di aggiungere un campo RouteMatch al proto HTTPFilter e modificare il gestore della catena di filtri per controllare la corrispondenza della route prima del dispatch.
Tech stack
cpp
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35