envoyproxy/envoy

Use StringMatcher for route path and header matching

Aperta

#8392 aperta il 26 set 2019

Vedi su GitHubGuida contributor
 (7 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
api/v3help wanted

Metriche repository

Star
 (27.997 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

As per #next-major-version comments in https://github.com/envoyproxy/envoy/blob/master/api/envoy/api/v2/route/route.proto. We should ideally do this in v2 with new fields and then deprecate the existing matchers, allowing for auto-deletion in v3.

Guida contributor

Direzione di ricerca
Esamina il file route.proto indicato nel problema, identifica i corrispondenti di stringhe regex esistenti e aggiungi nuovi campi StringMatcher per la corrispondenza del percorso e dell'intestazione. Depreca i vecchi campi. Assicura la compatibilità con v2 e pianifica la rimozione in v3. Testa le modifiche.
Tech stack
cpp
Dominio
backendnetworking
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65