envoyproxy/envoy

Investigate is it okay to kill the connection when the path includes `#`

Open

#10.033 aberto em 12 de fev. de 2020

Ver no GitHub
 (0 comments) (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

Issue Template

Title: Investigate is it okay to kill the connection when the path includes #

Description: Quote from the comment in https://github.com/envoyproxy/envoy/pull/10010#discussion_r378329481:

Note: # should be never sent on the wire (at least in theory), so we might want to kill such connections... but this would require investigation to make sure that this wouldn't kill some popular but broken clients.

Guia do colaborador

Direção de pesquisa
Revise a discussão no PR #10010, especialmente o comentário sobre '#'. Verifique se algum cliente envia caminhos com '#' e avalie o impacto de encerrar tais conexões. Examine o código de manipulação de conexões existente no Envoy para determinar onde adicionar a verificação.
Pilha de tecnologia
cpp
Domain
backendsecurity
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.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
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.
Principalmente claro
Prerequisites
C++HTTPproxy internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30