envoyproxy/envoy

AsyncClientFactory potentially calls ClusterManager::clusters() outside main thread

Open

#15.458 aperta il 12 mar 2021

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

Metriche repository

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

Descrizione

Title: AsyncClientFactory potentially calls ClusterManager::clusters() outside main thread

Description: An example of this is grpc access logger.

Another example is a filter implementation similar to ext_authz which does not skip static cluster checks (none exist in the open source codebase that I can find).

Repro steps: n/a

Admin and Stats Output: n/a

Config: n/a

Logs: see comment

Call Stack: n/a

related: #15447 #15227 #15449

Guida contributor

Direzione di ricerca
Esamina il logger di accesso grpc e la configurazione ext authz per verificare che ClusterManager::clusters() venga chiamato al di fuori del thread principale. Cerca asserzioni di controllo del thread mancanti.
Tech stack
cpp
Dominio
backend
Tipo issue
Bug
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40