envoyproxy/envoy

Refactor ext_proc client code to remove dynamic_cast

Open

#37 904 ouverte le 7 janv. 2025

Voir sur GitHub
 (1 commentaire) (0 réactions) (1 assigné)C++ (5 373 forks)batch import
area/ext_prochelp wantedtech debt

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Examinez la discussion dans la PR #35740 pour comprendre le contexte du dynamic cast. Étudiez la hiérarchie des classes dans source/extensions/filters/http/ext proc/ pour identifier un remplacement approprié (par exemple, une méthode virtuelle). Ensuite, implémentez le refactoring en remplaçant le dynamic cast à la ligne 30 de client impl.cc par un appel polymorphe ou un autre modèle de conception. Assurez vous qu'il n'y ait aucun changement de comportement.
Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75