-
Notifications
You must be signed in to change notification settings - Fork 1
implement users #23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
actually – it makes more sense to plug into an existing platform; i.e. use twitter to authenticate/user accounts. look at flask-social, which is built on flask-security. |
there still needs to be a user model, to keep track of settings and user-specific data. to keep flexibility for other accounts, need to also keep track of which service the token belongs to. so, need to save:
|
i'm having problems with flask-social which depends on python-oauth2. unfortunately, python-oauth2 doesn't work with python 3.3 it seems.
|
hm we may have to roll our own with https://github.com/lepture/flask-oauthlib |
foundational work here: |
I started adding in the oauth flows for authentication via google, twitter, and facebook. Haven't yet had a chance to test them: |
closed via 61c8f4d |
@ferristseng when you get a chance can you take a look and let me know what you think? mainly at the encryption of oauth token secrets/where those config values are stored. do you think they should be in |
it's probably better the way you have it. the only advantage to putting it in |
using flask-security
necessary for implementing oauth (see #22)
The text was updated successfully, but these errors were encountered: