envoyproxy/envoy

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

Open

#15.458 aberto em 12 de mar. de 2021

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/cluster_managerbughelp wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Examine o logger de acesso grpc e a configuração ext authz para verificar se ClusterManager::clusters() está sendo chamado fora da thread principal. Procure por asserções de verificação de thread ausentes.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40