Missing `Http2ServerResponse.setHeaders()`
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 122k
- Forks
- 37.3k
- Ø Merge
- 4 T. 2 Std.
- Gemergte PRs (30 T.)
- 283
Beschreibung
What is the problem this feature will solve?
Make it possible to use response.setHeaders() on both old HTTP response objects and ones coming from node:http2.
BTW I wasn't sure if this should be a "feature request" or a "bug."
What is the feature you are proposing to solve the problem?
Add the method http2.Http2ServerResponse.setHeaders() to have the same API as http.OutgoingMessage.setHeaders().
See https://nodejs.org/api/http.html#outgoingmessagesetheadersheaders.
What alternatives have you considered?
- Avoiding using
setHeaders()at all. - Monkey-patching
Http2ServerResponse. - Just writing two versions of my own set-headers code, on the assumption that, when available,
setHeaders()is the most efficient option.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem Einstiegspunkt node:http2 Http2ServerResponse und vergleiche seine API mit http.OutgoingMessage.setHeaders(), wobei die verlinkte HTTP-Dokumentation als Referenz für das Verhalten dient. Erledigt bedeutet, dass Http2ServerResponse setHeaders() mit derselben API und demselben Verhalten wie die vorhandenen HTTP-Antwortobjekte bereitstellt und eine angemessene Abdeckung für die neue Methode vorhanden ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, node.js
- Bereich
- api, backend
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 52/100