envoyproxy/envoy

Refactor ext_proc client code to remove dynamic_cast

Aperta

#37.904 aperta il 7 gen 2025

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

Metriche repository

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

Descrizione

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