usebruno/bruno

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

Open

#326 geöffnet am 4. Okt. 2023

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.403 Forks)batch import
good first issuehelp wantedmodule-request

Repository-Metriken

Stars
 (43.787 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 6T 21h) (96 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche die Bruno Codebasis, um herauszufinden, wo das Parsen der HTTP Anfrage stattfindet, wahrscheinlich in einem Modul, das den eingehenden Anfragetext verarbeitet. Der Fehler deutet darauf hin, dass ein benutzerdefinierter Parser verwendet wird. Untersuche den 'send http request' Handler, um die Stelle zu finden, an der das Header Parsing durchgeführt wird, und ändere ihn so, dass LF gemäß RFC7230 Abschnitt 3.5 als gültiges Zeilenendezeichen akzeptiert wird. Erwäge Tests mit beiden Varianten CRLF und LF.
Tech Stack
javascript
Domain
backendapi
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
HTTPNode.js
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60