usebruno/bruno

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

Open

#326 创建于 2023年10月4日

在 GitHub 查看
 (2 评论) (0 反应) (0 负责人)JavaScript (43,787 star) (2,403 fork)batch import
good first issuehelp wantedmodule-request

描述

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.

贡献者指南

技术栈
javascriptnodejs
领域
backendapi
议题类型
feature
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 hours
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
fresh
清晰度议题是否清楚说明期望改动、验收标准和下一步。
clear
前置要求
无
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
80
研究方向
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.
[Feature Request] Support LF line breaks as well as CRLF line breaks · usebruno/bruno#326 | Good First Issue