envoyproxy/envoy

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

Open

#15.458 geöffnet am 12. März 2021

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/cluster_managerbughelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie den grpc Zugriffslogger und die ext authz Konfiguration, um zu überprüfen, ob ClusterManager::clusters() außerhalb des Hauptthreads aufgerufen wird. Suchen Sie nach fehlenden Thread Überprüfungs Assertionen.
Tech Stack
cpp
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40