envoyproxy/envoy

Router: Audit ASSERTs

Open

#14.961 aberto em 5 de fev. de 2021

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/securityhelp 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

This issue describes a scan of ASSERTs in the router filter and recommendations to change some: https://gist.github.com/asraa/859a41a8f5f252d35da58c9ef7502230

Those marked with a verdict have been audited or unknown if they need a pair of eyes.

Guia do colaborador

Direção de pesquisa
Revise o gist em https://gist.github.com/asraa/859a41a8f5f252d35da58c9ef7502230, que descreve a auditoria de ASSERT. O arquivo principal a ser modificado é source/common/router/router.cc, com as linhas específicas: 276, 566 e 653. A tarefa envolve adicionar uma verificação para cabeçalhos de esquema durante o processamento do filtro e auditar outros ASSERTs conforme marcado. Inspecione o código ao redor dessas linhas para entender a lógica atual do assert e aplicar as alterações recomendadas. Não há PRs vinculados, portanto a implementação é independente.
Pilha de tecnologia
cpp
Domain
backendinfrastructure
Tipo Issue
Refatoração
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.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Envoy internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40