usebruno/bruno

[Feature Request] Support LF line breaks as well as CRLF line breaks

Open

#326 aperta il 4 ott 2023

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)JavaScript (2403 fork)batch import
good first issuehelp wantedmodule-request

Metriche repository

Star
 (43.787 star)
Metriche merge PR
 (Merge medio 6g 21h) (96 PR mergiate in 30 g)

Descrizione

Currently, when a client sends LF instead of CRLF as a newline character, Bruno errors out with the following message:

Error invoking remote method 'send-http-request': Error: Parse Error: Missing expected CR after header value

Insomnia does support LF newlines, and I think Bruno should as well; RFC7230 Section 3.5 also allows for this:

Although the line terminator for the start-line and header fields is the sequence CRLF, a recipient MAY recognize a single LF as a terminator and ignore any preceding CR.

Guida contributor

Direzione di ricerca
Esamina la codebase di Bruno per individuare dove avviene il parsing delle richieste HTTP, probabilmente in un modulo che gestisce il testo della richiesta in arrivo. L'errore suggerisce che viene utilizzato un parser personalizzato. Indaga il gestore 'send http request' per trovare dove viene effettuato il parsing degli header e modificalo per accettare LF come terminatore di riga valido secondo RFC7230 Sezione 3.5. Considera di testare con entrambe le varianti CRLF e LF.
Tech stack
javascript
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
HTTPNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60