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

OAuth Providers array in user object #86

Closed
maximousblk opened this issue Apr 6, 2021 · 4 comments
Closed

OAuth Providers array in user object #86

maximousblk opened this issue Apr 6, 2021 · 4 comments
Projects

Comments

@maximousblk
Copy link

Feature request

Is your feature request related to a problem? Please describe.

In app_metadata.provoder in the user object, currently we can only get what provider did the user signed up with and not all the provides that the user has connected.

Describe the solution you'd like

Instead of providing only the initial OAuth Provider, provide an array of all the providers user has used in the past.

{
  // ...
  "app_metadata": {
-   "provider": "github"
+   "providers": ["github", "gitlab"]
  },
  // ...
}

This could be a massive breaking change, so here's an alternative solution:

Provide the current OAuth Provider and Dev's can record it on the application side.

Describe alternatives you've considered

N/A

Additional context

This can be used to show which OAuth Provider the user has connected and if they want to disconnect a provider or connect more, similar to the following:

Screenshot_20210406-124145_Chrome
ref: GitLab profile settings

@kiwicopple
Copy link
Member

Nice - seems like a good idea. I'll shift this to our GoTrue repo 👍

@kiwicopple kiwicopple transferred this issue from supabase/supabase Apr 14, 2021
@maximousblk
Copy link
Author

maximousblk commented Apr 14, 2021

I'd like to add that it would be nice if email:pass and email-link are specified separately so that in case an app needs password to be setup (login on cli and similar), but user signed up using magic links we can prompt the user to set it up.

@maximousblk
Copy link
Author

Any updates on this?

@bnjmnt4n
Copy link
Contributor

This is addressed by #208 (and relevantly, #240).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Tools
Triage
Development

No branches or pull requests

3 participants