envoyproxy/envoy

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

Open

#15 458 ouverte le 12 mars 2021

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/cluster_managerbughelp wanted

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: 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

Guide contributeur

Direction de recherche
Examinez le journal d'accès grpc et la configuration ext authz pour vérifier que ClusterManager::clusters() est appelé en dehors du thread principal. Recherchez les assertions de vérification de thread manquantes.
Stack technique
cpp
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40