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

implemented 4 flow of authentication #30

Open
wants to merge 67 commits into
base: develop
Choose a base branch
from

Conversation

yoh1496
Copy link
Collaborator

@yoh1496 yoh1496 commented Jul 9, 2020

I implemented 4 flow of authentication.

  1. with ROPC
  2. with iframe (oauth2)
  3. with new tab (oauth2)
  4. with current tab (oauth2)

The first flow is not app auth, so you shouldn't use.

I'm thinking how to choose one of them which developer would like to use.

image

@yoh1496 yoh1496 changed the title implemented 4 flow of authentication 【WIP】implemented 4 flow of authentication Jul 9, 2020
@yoh1496 yoh1496 self-assigned this Jul 9, 2020
tochi-y and others added 3 commits July 10, 2020 17:42
@dixonsiu dixonsiu requested review from j1n6 and tochi-y November 26, 2020 01:54
@dixonsiu dixonsiu added the enhancement New feature or request label Nov 26, 2020
@yoh1496 yoh1496 changed the title 【WIP】implemented 4 flow of authentication implemented 4 flow of authentication Jun 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants