browserify / browserify/http-browserify

Don't `require('buffer')`

Open
#22 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
245
Forks
104
PR merge metrics
No merged PRs in 30d

Description

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.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by tracing where http-browserify requires buffer and how the auth parameter is handled. Compare the stated options—documenting that auth is unsupported or using a base64 library—and inspect the existing dependency setup before choosing a direction. Done means the buffer dependency is no longer unnecessarily required and the auth behavior is explicit and covered by the project’s existing checks.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
web-dev
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.