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

fix: replace deprecated flask-oauthlib with authlib #1411

Merged
merged 3 commits into from
Jul 3, 2020

Conversation

kdwyer
Copy link
Contributor

@kdwyer kdwyer commented Jun 23, 2020

As documented in #748, flask-oauthlib has been deprecated in favour of authlib. In this issue, authlib being licensed under AGPL was a blocker, but Authlib is now licensed under BSD.

This patch updates code, docs and example code to use authlib instead of flask-oauthlib.

I've tested login against all the supported OAuth providers except LinkedIn, for which I don't have the necessary credentials.

kdwyer added 3 commits June 23, 2020 17:55
Flask-oauthlib has been deprecated by its maintainer; authlib is the
recommended successor.
@kdwyer kdwyer force-pushed the authlib-integration branch from d3b763b to c0caa91 Compare June 23, 2020 16:58
@dpgaspar dpgaspar changed the title Fix: replace deprecated flask-oauthlib with authlib fix: replace deprecated flask-oauthlib with authlib Jul 3, 2020
Copy link
Owner

@dpgaspar dpgaspar left a comment

Choose a reason for hiding this comment

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

Awesome!

@dpgaspar dpgaspar merged commit 85567d5 into dpgaspar:master Jul 3, 2020
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.

2 participants