envoyproxy/envoy

Refactor ext_proc client code to remove dynamic_cast

Open

#37.904 aperta il 7 gen 2025

Vedi su GitHub
 (1 commento) (0 reazioni) (1 assegnatario)C++ (5373 fork)batch import
area/ext_prochelp wantedtech debt

Metriche repository

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

Descrizione

Title: Refactor ext_proc client code to remove dynamic_cast

Description:

Remove this dynamic_cast here:

https://github.com/envoyproxy/envoy/blob/29428f956761918e7112183af50f969b099196f1/source/extensions/filters/http/ext_proc/client_impl.cc#L30

For more details please check: https://github.com/envoyproxy/envoy/pull/35740#discussion_r1781527990

Guida contributor

Direzione di ricerca
Rivedere la discussione nella PR #35740 per comprendere il contesto del dynamic cast. Esaminare la gerarchia delle classi in source/extensions/filters/http/ext proc/ per identificare una sostituzione adeguata (ad esempio, un metodo virtuale). Quindi implementare il refactoring sostituendo il dynamic cast alla riga 30 di client impl.cc con una chiamata polimorfica o un diverso pattern di progettazione. Assicurarsi che non vi siano cambiamenti nel comportamento.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75