[Feature Request] Support LF line breaks as well as CRLF line breaks · usebruno/bruno#326 | Good First Issue

(2 评论) (0 反应) (0 负责人)JavaScript (2,403 fork)batch import

good first issuehelp wantedmodule-request

仓库指标

Star: (43,787 star)
PR 合并指标: (平均合并 6天 21小时) (30 天内合并 96 个 PR)

描述

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.

贡献者指南

研究方向: 检查 Bruno 代码库中 HTTP 请求解析发生的位置，很可能在处理传入请求文本的模块中。错误提示正在使用自定义解析器。调查 'send http request' 处理程序，找到进行标头解析的位置，并根据 RFC7230 第 3.5 节修改它以接受 LF 作为有效的行终止符。考虑使用 CRLF 和 LF 两种变体进行测试。
技术栈: javascript
领域: backendapi
议题类型: 功能
难度: 3
预计时间: 1-2 天
活动状态: 活跃
清晰度: 清晰
前置要求: HTTPNode.js
新手友好度: 60