coinbase / coinbase-python

DEPRECATED — Coinbase Python API
Apache License 2.0
528 stars 217 forks source link

Should support OAuth setup for users w/o access, refresh token #1

Closed pgorsira closed 9 years ago

pgorsira commented 9 years ago

The old coinbase-python had functions for steps involved in the OAuth process. This library should support more than just making calls as the already-OAuthed user.

pgorsira commented 9 years ago

Nvm, appears I was mistaken, the other library just has a nice example included that makes use of other 3rd party libs: https://github.com/sibblegp/coinbase_python/blob/master/coinbase_oauth2/__init__.py

jorilallo commented 9 years ago

@pgorsira As implementing OAuth2 flow is pretty specific to web framework of your choosing we decided to leave it out in this library. We'll be including more tools to our documentation etc to help.

I would highly recommend checking out python-social-auth. I have used it in the past and it's pretty flexible. Otherwise, implementing it from scratch doesn't take too much effort as long as you remember to supply state param to protect your users from CSRF attacks.