nodejs/node

No content length on DELETE and OPTIONS

Open

#27 880 ouverte le 26 mai 2019

Voir sur GitHub
 (26 commentaires) (0 réactions) (0 assignés)JavaScript (35 535 forks)batch import
confirmed-bughelp wantedhttp

Métriques du dépôt

Stars
 (117 218 stars)
Métriques de merge PR
 (Merge moyen 18j 17h) (219 PRs mergées en 30 j)

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?

Guide contributeur

Direction de recherche
Examiner la gestion manquante de Content Length pour les méthodes DELETE et OPTIONS dans http outgoing.js. Suivez la logique autour de la ligne 349 dans le code source de Node.js et assurez la cohérence avec le correctif dans le commit 8a24a1e18f.
Stack technique
javascript
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
GitNode.js internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60