Missing `Http2ServerResponse.setHeaders()`
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 122k
- Forks
- 37.3k
- Merge moyen
- 4 j 2 h
- PR mergées (30 j)
- 283
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par le point d’entrée node:http2 Http2ServerResponse et comparez son API à celle de http.OutgoingMessage.setHeaders(), en utilisant la documentation HTTP liée comme référence comportementale. Le travail est considéré comme terminé lorsque Http2ServerResponse expose setHeaders() avec la même API et le même comportement que les objets de réponse HTTP existants, avec une couverture appropriée pour la nouvelle méthode.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, node.js
- Domaine
- api, backend
- Type d'issue
- Fonctionnalité
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- Calme
- Clarté
- Plutôt claire
- Accessibilité débutants
- 52/100