browserify / browserify/http-browserify
Don't `require('buffer')`
- 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