valyala/fasthttp

Request should not fail on small ReadBufferSize

Fechada

#1.873 aberto em 24 de set. de 2024

Ver no GitHub
 (4 comentários) (0 reação) (0 responsável)Go (1.755 forks)batch import
help wantedwontfix

Métricas do repositório

Stars
 (21.741 estrelas)
Métricas de merge de PR
 (Mesclagem média 3d) (44 fundiu PRs em 30d)

Description

Can we not fail request if the req/resp header size is greater than the read buffer size.

I can see a bufio bug was mentioned for tryRead function but looks like it's resolved now. Can we have the above mentioned behaviour?

Also, some callback could also be configured to inform that buffer size was low. Library users could emit metrics and increase the buffer size appropriately to have best balance between memory usage and CPU usage.

@valyala Could you review the request?

Guia do colaborador

Direção de pesquisa
Primeiro, leia o arquivo header.go do fasthttp, especialmente a função tryRead na linha 2128 e a lógica do buffer ao redor. Entenda como ReadBufferSize afeta a análise da solicitação. O problema faz referência a um bug do Go (golang/go#14121) já resolvido; investigue se essa resolução pode ser aplicada ao fasthttp. Além disso, considere implementar um mecanismo de callback para informar os usuários quando o tamanho do buffer estiver baixo, permitindo que eles se ajustem. Verifique os comentários existentes para qualquer feedback dos mantenedores ou contexto adicional.
Pilha de tecnologia
gorest api
Domain
backendperformance
Tipo Issue
Funcionalidade
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
Understanding of GoFamiliarity with fasthttp's bufferingReading header.go file
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
30