browserify / browserify/http-browserify

Don't `require('buffer')`

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

説明

The buffer implementation is a lot of code and http-browserify does not really need it to work. It is only needed for `auth` parameter to bas64 encode it.

Could it be document that the `auth` parameter is not supported or changed to depend on a base64 library or? Eg. `base64-js` which `buffer-browserify` depends on.

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

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

調査の方向性

まず、http-browserify が buffer を require している箇所と、auth パラメータの扱いを追跡します。提示されている選択肢(auth がサポートされていないことを文書化するか、base64 ライブラリを使用するか)を比較し、方向性を決める前に既存の依存関係の設定を確認します。buffer 依存関係が不要に require されなくなり、auth の動作が明示され、プロジェクトに既存のチェックでカバーされれば完了です。

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

評価

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

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

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