Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't require('buffer') #22

Open
tellnes opened this issue Aug 8, 2013 · 3 comments
Open

Don't require('buffer') #22

tellnes opened this issue Aug 8, 2013 · 3 comments

Comments

@tellnes
Copy link
Contributor

tellnes commented Aug 8, 2013

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.

@feross
Copy link
Member

feross commented Dec 5, 2013

Where do you see require('buffer')? It looks like buffer is not directly required by http-browserify but rather by stream, which is needed to make the request and response objects be streams. Streams operate on Buffers, so it's required.

@tellnes
Copy link
Contributor Author

tellnes commented Sep 9, 2014

When I filled this was buffer directly required by http-browserify. Probably did someone resolve it and now, it has been reintroduced by stream.

@feross
Copy link
Member

feross commented Jul 8, 2015

This issue should be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants