browserify / browserify/http-browserify

case insensitive headers

オープン
#91 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
245
フォーク
104
PR マージ指標
30日以内にマージされた PR はありません

説明

I understand HTTP header names are case-insensitive (RFC 2616). I'm currently interacting with an API that does not follow protocol, it works via node's https but not browserify's.

If you want complete compatibility with node, I'd say remove the case insensitivity. Otherwise, if you want to just follow the spec, I understand.

In the meantime, I've been able to get my API working, by modifying the browserify source code to remove the tolowercase for the headers

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

まず、ヘッダー名を小文字に変換する Browserify のソースコードを見つけ、Node の https 実装との動作を比較します。非準拠の API との互換性が必要なのか、それとも RFC に準拠した大文字と小文字を区別しないヘッダーが必要なのかを明確にし、その後、選択した動作に対する集中的なカバレッジを追加します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript
領域
networking
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。