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

Switch to Authlib #1434

Merged
merged 9 commits into from Feb 14, 2018

Conversation

Projects
None yet
2 participants
@noirbizarre
Member

noirbizarre commented Feb 13, 2018

This PR:

  • Switch to Authlib instead of the unmaintained OAuthlib/Flask-OAuthlib
  • Ensure refresh tokens are working
  • Makes OAuth 2 implementation support all grantypes
  • Support token revocation
  • Makes internal client allowed to bypass authorization screen
  • Ports test to Pytest
  • Ensures tests cover all cases

@noirbizarre noirbizarre added this to the 1.3.0 milestone Feb 13, 2018

@noirbizarre noirbizarre self-assigned this Feb 13, 2018

@noirbizarre noirbizarre requested a review from opendatateam/etalab Feb 13, 2018

Show outdated Hide outdated setup.cfg
Show outdated Hide outdated udata/migrations/2018-02-12-authlib.js

@noirbizarre noirbizarre changed the title from WIP: Switch to Authlib to Switch to Authlib Feb 14, 2018

@noirbizarre noirbizarre requested a review from opendatateam/etalab Feb 14, 2018

@abulte

abulte approved these changes Feb 14, 2018

@noirbizarre noirbizarre merged commit 19cf92d into opendatateam:master Feb 14, 2018

3 checks passed

ci/circleci: assets Your tests passed on CircleCI!
Details
ci/circleci: dist Your tests passed on CircleCI!
Details
ci/circleci: python Your tests passed on CircleCI!
Details

@noirbizarre noirbizarre deleted the noirbizarre:authlib branch Feb 14, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment