-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
🚀 Feature: SMS login (and Magic link) is not consitent with Email login #6154
Comments
@Megamannen, thanks for raising this issue! 🙏🏼 I would suggest setting the Users Limit to 1: This should prevent new accounts from being created. Then, you can use a server SDK and API Key to create users (which bypasses this limit). Does that solve your use case? |
No, since I don't have any way of knowing the ID/UserId, or am I missing something? Feels the current function can only be used for registration and not login |
What the OP means is that Something like: |
You are correct in my need. But also remember that it's impossible to provide the matching [USER_ID] when logging in |
@Megamannen, if you set the user limit to some low number and someone calls
appwrite/app/controllers/api/account.php Line 920 in 891099c
appwrite/app/controllers/api/account.php Line 1258 in 891099c
You don't need to provide a matching user id when just logging in. that user id parameter is only used during account creation. Otherwise, it's ignored. |
In that case it will work in my use case, thank you! |
🔖 Feature description
Right now Create Email Session is the odd duck. Since it doesn't create the user if it doesn't exist.
I'm mainly focusing on the Create Phone session
Possible solution is to accept a "null" userID which can mean "existing user only"
🎤 Pitch
I don't want user registration, but I want to allow signing in with phone or magic link.
Right now that is not possible since the API will create the user if the phone/email doesn't exist.
Probably a fairly common use case?
👀 Have you spent some time to check if this issue has been raised before?
🏢 Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: