nodejs/node

No content length on DELETE and OPTIONS

Offen

#27.880 geöffnet am 26.05.2019

Auf GitHub ansehenContributor Guide
 (26 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (35.535 Forks)batch import
confirmed-bughelp wantedhttp

Repository-Metriken

Stars
 (117.218 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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?

Contributor Guide

Research-Richtung
Untersuchen Sie die fehlende Content Length Behandlung für DELETE und OPTIONS Methoden in http outgoing.js. Verfolgen Sie die Logik um Zeile 349 im Node.js Quellcode und stellen Sie die Konsistenz mit dem Fix in Commit 8a24a1e18f sicher.
Tech Stack
javascript
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
GitNode.js internals
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60