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

FR: possibility to deactivate SignUp when signing in with email link #2418

Open
fabianmarcus opened this issue Dec 8, 2019 · 3 comments
Open

Comments

@fabianmarcus
Copy link

@fabianmarcus fabianmarcus commented Dec 8, 2019

[REQUIRED] Describe your environment

Firebase Auth, latest version.

[REQUIRED] Describe the problem

sendSignInLinkToEmail(email, settings) does not only sign the user in but also registers him when his email does not exist yet.

The problem is that this behavior could undermine an extra registration process, in which the user has to specify more data than just the email.

Steps to reproduce:

Use sendSignInLinkToEmail(email, settings) with a user email that has not been registered yet.

FEATURE IDEA

The automatic sign up can be disabled in the firebase console. When a user tries to login with an email that has not been registered yet firebase returns an error which can be catched.

Relevant Code:

https://stackoverflow.com/questions/58652045/why-does-sendsigninlinktoemailemail-settings-not-check-whether-the-given-emai

@rommelpe

This comment has been minimized.

Copy link

@rommelpe rommelpe commented Dec 9, 2019

Thanks for filing a feature request, @fabianmarcus. I created an internal FR (b/145924653) for this.

@rommelpe

This comment has been minimized.

Copy link

@rommelpe rommelpe commented Dec 9, 2019

I got feedback from Auth folks that we actually support the option to disable sign up but this is currently only available via the Cloud Console (for Identity Platform projects only).

We have plans to expose this in the Firebase Console yet can't share any timelines or specifics for now.

@fabianmarcus

This comment has been minimized.

Copy link
Author

@fabianmarcus fabianmarcus commented Dec 12, 2019

Mhm... Then I'm looking forward to hear some news about this!
Thank you anyways

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.