nodejs/node

No content length on DELETE and OPTIONS

Open

#27.880 aberto em 26 de mai. de 2019

Ver no GitHub
 (26 comments) (0 reactions) (0 assignees)JavaScript (35.535 forks)batch import
confirmed-bughelp wantedhttp

Métricas do repositório

Stars
 (117.218 stars)
Métricas de merge de PR
 (Mesclagem média 18d 17h) (219 fundiu PRs em 30d)

Description

Looking through https://github.com/nodejitsu/node-http-proxy/commit/8a24a1e18f384d29a125eda1d2311bdd8ec66871 and reading through the node code. I do believe we have an unhandled case.

I believe we are missing something here: https://github.com/nodejs/node/blob/master/lib/_http_outgoing.js#L349

To ensure content-length and transfer-encoding are correct.

Am I on to something or should I drop this?

Guia do colaborador

Direção de pesquisa
Investigue a manipulação ausente de Content Length para os métodos DELETE e OPTIONS em http outgoing.js. Rastreie a lógica em torno da linha 349 no código fonte do Node.js e garanta a consistência com a correção no commit 8a24a1e18f.
Pilha de tecnologia
javascript
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.
Principalmente claro
Prerequisites
GitNode.js internals
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60