Skip to content
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

Reworked login system #861

Merged
merged 5 commits into from May 10, 2020
Merged

Reworked login system #861

merged 5 commits into from May 10, 2020

Conversation

RAnders00
Copy link
Contributor

@RAnders00 RAnders00 commented May 9, 2020

Login is now (imo) nicer and more maintainable

  • Remove dependency on deprecated flask-oauthlib (Closes flask-oauthlib has been deprecated #816)
  • added csrf validation to login process
  • Makes login code compatible with upcoming changes to helix authentication
  • Login error page now tells you what went wrong
  • We don't store the twitch access token in the session cookie for no reason anymore
  • We no longer request the scope to read the user's email by default, the app works just fine with an empty list of scopes.

Pull request checklist:

  • CHANGELOG.md was updated, if applicable
  • Documentation in docs/ or install-docs/ was updated, if applicable

@RAnders00 RAnders00 requested a review from pajlada May 9, 2020 15:09
@ALazyMeme
Copy link
Member

Any chance you'd want to have a look at this as well? :D #450
Since it's something to do with the login that you're fixing.

@RAnders00
Copy link
Contributor Author

That's not really login related, and does not fit into this PR at all. We can do it afterwards, and it would be just as easy to do as before this PR.

Copy link
Member

@pajlada pajlada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

inbana.pjg

@pajlada pajlada merged commit 821500c into master May 10, 2020
@pajlada pajlada deleted the randers00-reworked-login branch May 10, 2020 11:33
RAnders00 added a commit that referenced this pull request May 16, 2020
I forgot to commit that as part of #861
@RAnders00 RAnders00 mentioned this pull request May 16, 2020
2 tasks
RAnders00 added a commit that referenced this pull request May 16, 2020
I forgot to commit that as part of #861
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

flask-oauthlib has been deprecated
3 participants