envoyproxy/envoy

Move HCM user agent stats functionality into an optional filter

Open

#10.195 aperta il 27 feb 2020

Vedi su GitHub
 (11 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/httpbeginnerhelp wanted

Metriche repository

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

Descrizione

Right now we have user agent detection built directly into the HCM: https://github.com/envoyproxy/envoy/blob/master/source/common/http/user_agent.h

This code is very old and at this point should not be required to be built into the HCM. We should move this code into an optional filter.

Guida contributor

Direzione di ricerca
Esamina i file user agent.h e user agent.cc esistenti in source/common/http per comprendere l'implementazione attuale. Quindi, crea un nuovo filtro che fornisca la stessa funzionalità come filtro opzionale, seguendo il modello degli altri filtri nel repository. Assicurati che possa essere abilitato/disabilitato tramite configurazione.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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++Envoy development basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65