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

Deleting account without password [oauth only setups] #137

Open
LouisDeconinck opened this issue Aug 30, 2024 · 3 comments
Open

Deleting account without password [oauth only setups] #137

LouisDeconinck opened this issue Aug 30, 2024 · 3 comments

Comments

@LouisDeconinck
Copy link
Contributor

When only using social OAuth the user does not have a password. When trying to delete an account you have to give a password as confirmation. Keep the confirmation but remove the need for a password.

Additionally, to remove email auth, you can set within the Auth component from @supabase/auth-ui-svelte: onlyThirdPartyProviders={true} This will remove the email and password fields, which will otherwise show up, even when disabling email auth within Supabase.

@scosman
Copy link
Contributor

scosman commented Aug 30, 2024

Good find. Want to make a PR?

We need some way of making sure you can’t delete an account with UI access for 5 seconds.

see how we handle password reset. We require the “amr” to be “recovery” to verify they own the email. Can probably do the same here? Not sure about if it works with oauth but I think it should.

@scosman scosman changed the title Deleting account without password Deleting account without password [oauth only setups] Aug 31, 2024
@scosman
Copy link
Contributor

scosman commented Aug 31, 2024

Edit: you can always set a password, even for oauth accounts. So not impossible. But not smooth either.

@LouisDeconinck
Copy link
Contributor Author

What does AMR stand for and why must we prevent an account to be deleted that fast? Wouldn't simply having a confirmation button be sufficient?

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

No branches or pull requests

2 participants