OAuth2 Support - Githubissues

MediaMath / t1-python

Python SDK for MediaMath Platform APIs

https://developer.mediamath.com/

Apache License 2.0

18 stars 28 forks source link

OAuth2 Support #62

Closed pswaminathan closed 8 years ago

pswaminathan commented 9 years ago

Add support for OAuth2 as an authentication method
Refactor auth in general to happen within Connection, using Service more as a mechanism for getting a session in Connection live.
[x] Add tests
[x] Fix breaking unit tests (which I think are breaking because session now passes in API key on every request)
[x] Add documentation to README.rst

pswaminathan commented 8 years ago

@FodT can you take a look? Rebased on master and has unit tests. Because setting up an OAuth2 session requires manual work (going through the web app flow), I don't know that it makes sense to add this to the acceptance tests. But perhaps some way.

FodT commented 8 years ago

We could look into providing a template app, and writing some acceptance test in capybara?