authgear / authgear-server

Open source alternative to Auth0 / Firebase Auth
https://www.authgear.com
Apache License 2.0
81 stars 37 forks source link

Linking account in AuthUI and Authflow #4103

Closed fungc-io closed 5 months ago

fungc-io commented 6 months ago

If OAuth conflicts with OAuth, the screen should show the other social login buttons too.

Original Account Incoming Account Possible actions
Email OAuth (Google, Facebook, etc..) 1. Error: Remind the user to login with Email or 2. Login with the original authenticator (password/OTP) to link the account
OAuth (Google, Facebook, etc..) Another OAuth (Google, Facebook, etc..) 1. Error: Remind the user to login with OAuth or 2. Login with the original OAuth to link the account
OAuth (Google, Facebook, etc..) Email Error: Remind the user to login with OAuth or

~In the 3rd case, if the original account is OAuth and the user enters the email to log in, the linking process will be complicated because the user needs to:~

  1. ~confirms they own the original OAuth account by logging in, and then~
  2. ~set up the authenticators like password or OTP.~

~We can just simply tell them to login with OAuth without linking~


Legacy issue:

linear[bot] commented 6 months ago

DEV-1195 Confirm merging account in AuthUI and Authflow

fungc-io commented 6 months ago

@tung2744 @newman @louischan, i've summarized the behavior of different combinations we discussed today in above table

fungc-io commented 6 months ago

In the product meeting today, instead of having different behaviour for incoming account = email/oauth, A better way should be always continue the signup flow after "login and link", and skip the step if the required authenticator already exists in the account.

This prevents that the linked authenticator cannot be used for login because the required authenticator is missing.

For example:

fungc-io commented 5 months ago

@tung2744 I've prepared a testing spec for the current phase of account linking:

Testing Spec for Account Linking

Treat it as acceptance criteria and i will share it with QA team for testing. Please see if this align with our plan?

tung2744 commented 5 months ago

Yes, I think it aligns with our expectation.