search

browserify / http-browserify

node's http module, but for the browser
MIT License
244 stars 110 forks source link

Suggested Setup in README Doesn't Work #67

Open codeviking opened 9 years ago

codeviking commented 9 years ago

When I dial this into the latest browserify:

var bundle = browserify({
    require : { http : 'http-browserify' }
});

I get an error saying Error: path must be a string.

Please update with correct configuration instructions!

skyhaikuteam commented 9 years ago

Did you ever manage to resolve this? I'm also experiencing the same issue. Wonder if the browserify api has changed fairly recently.

codeviking commented 9 years ago

@skyhaikuteam I did not. I ended up not using the package due to the issue.

deathcap commented 9 years ago

@codeviking @skyhaikuteam browserify now automatically uses http-browserify for http by default:

https://github.com/substack/node-browserify/blob/master/lib/builtins.js

exports.http = require.resolve('http-browserify');

the config from the readme is no longer needed (and should probably be deleted):