envoyproxy/envoy

HTTP/1.1 codec: Audit ASSERTs

Open

#14.959 aberto em 5 de fev. de 2021

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)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/1.1 codec and recommendations to change some: https://gist.github.com/asraa/9d5d60b0af53dc88ce39d8cfd71ef12a

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/9d5d60b0af53dc88ce39d8cfd71ef12a para ver os resultados da auditoria. Em seguida, implemente alterações para as duas condições acionáveis nas linhas 341 e 381 em source/common/http/http1/codec impl.cc. Para as linhas marcadas como UNKNOWN (141, 804, 1148), analise as asserções para determinar se elas podem ser acionadas por entrada não confiável; se sim, converta as para o tratamento de erro apropriado. Os mantenedores não forneceram esclarecimentos adicionais nos comentários.
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.
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 development setup
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40