envoyproxy/envoy

HTTP/2 codec: Audit ASSERTs

Open

#14.960 aberto em 5 de fev. de 2021

Ver no GitHub
 (0 comments) (0 reactions) (1 assignee)C++ (5.373 forks)batch import
area/httparea/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 HTTP/2 codec and recommendations to change some: https://gist.github.com/asraa/eb8b90a6398d202dd584cf403e3c6f3a

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 vinculado na issue para recomendações de auditoria. Examine as linhas ASSERT especificadas em source/common/http/http2/codec impl.cc (linhas 223, 387, 391, 750, 773, 1085). Determine se cada ASSERT pode ser acionado por entrada externa e decida se deve alterá lo para um mecanismo de tratamento de erro mais robusto (por exemplo, retornar erro ou ENVOY LOG). A issue não tem comentários ou PRs; verifique o status do responsável e o gist para quaisquer atualizações.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Pesquisa
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
4
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.
Principalmente claro
Prerequisites
C++HTTP/2
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30