Missing `Http2ServerResponse.setHeaders()`
Nessuno ha ancora preso questa issue.
- Lingua principale
- JavaScript
- Stelle
- 122k
- Fork
- 37.3k
- Merge medio
- 4g 2h
- PR unite (30g)
- 283
Descrizione
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.
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia dal punto di ingresso node:http2 Http2ServerResponse e confronta la sua API con http.OutgoingMessage.setHeaders(), usando la documentazione HTTP collegata come riferimento per il comportamento. Il lavoro è completato quando Http2ServerResponse espone setHeaders() con la stessa API e lo stesso comportamento degli oggetti di risposta HTTP esistenti, con una copertura adeguata per il nuovo metodo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- javascript, node.js
- Ambito
- api, backend
- Tipo di issue
- Funzionalità
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Tranquilla
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 52/100