usebruno/bruno

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

Open

#326 ouverte le 4 oct. 2023

Voir sur GitHub
 (2 commentaires) (0 réactions) (0 assignés)JavaScript (43 787 stars) (2 403 forks)batch import
good first issuehelp wantedmodule-request

Description

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.

Guide contributeur

Stack technique
javascriptnodejs
Domaine
backendapi
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80
Direction de recherche
Examine the Bruno codebase for where the HTTP request parsing occurs, likely in a module that handles incoming request text. The error suggests a custom parser is being used. Investigate the 'send http request' handler to find where header parsing is done and modify it to accept LF as a valid line terminator per RFC7230 Section 3.5. Consider testing with both CRLF and LF variations.