nodejs/node

No content length on DELETE and OPTIONS

Ouverte

#27 880 ouverte le 26 mai 2019

Voir sur GitHubGuide contributeur
 (26 commentaires) (0 réaction) (0 personne assignée)JavaScript (35 535 forks)batch import
confirmed-bughelp wantedhttp

Métriques du dépôt

Stars
 (117 218 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

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