search

jmdobry / disqus-node

http://disqus.com/api/ API bindings and CLI for NodeJS
http://disqus-node.pseudobry.com/
Apache License 2.0
15 stars 5 forks source link

Client-side support? #15

Closed matthewgertner closed 9 years ago

matthewgertner commented 9 years ago

I'd like to use your package for a client-side web app using Browserify. The problem is that you currently require the api_secret for all APIs, which precludes use on the client side unless you want to expose your key to the world. Some APIs (e.g. details) will work with the public key, and I guess that client-side OAuth can be performed without the secret key, right? Would it make sense to add support for this to your package?

jmdobry commented 9 years ago

disqus-node was made specifically for NodeJS. The CLI code would be dead weight in the browser, and I think the size of the browserified file would be unreasonable. I think it makes more sense for a client-side disqus client to be its own library, with its own made-for-the-browser dependencies.

matthewgertner commented 9 years ago

Fair enough.