search

desmondmorris / node-twitter

Client library for the Twitter REST and Streaming API's.
https://www.npmjs.com/package/twitter
MIT License
1.27k stars 237 forks source link

Replace the request dependency with lighter alternative #281

Open dandv opened 6 years ago

dandv commented 6 years ago

If this module is still maintained (#279), any plans to ditch request in favor of something lighter and ideally that works in the browser as well, like cross-fetch? That would also solve #252. CC @frizz925

frizz925 commented 6 years ago

Hopefully I wasn't too late for this. I'll start working on this issue on my fork by this weekend hopefully https://github.com/Frizz925/node-twitter/issues/1

desmondmorris commented 6 years ago

@dandv @Frizz925 this sounds like a great a idea. I have not looked into this too much, but I imagine replacing the oauth bits and some of the request sugar I am using, may get a little hairy. I will keep my eye out for a patch

dandv commented 6 years ago

@desmondmorris: thoughts on this rewrite using modern underlying libraries?

desmondmorris commented 6 years ago

hel