-
Notifications
You must be signed in to change notification settings - Fork 368
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
Create a supabase.auth.admin.getUserByEmail
function
#880
Comments
Hey yes this is a good suggestion. Though, we would probably use something like |
Hey, I am a new comer here, can I pick up this task? |
Hey @toff26, Sorry for the delayed reply - do feel free to take a stab at it and let us know if there are any queries. You'll need to update both Thanks! |
@J0 @hf i don't see an open PR shall i pick this up?
|
Leaving this remark here for easier reference, as this issue has already resolved. |
@Ganthology Thanks for the PR! We'll take a look shortly. |
Has anyone run the command recently / was anyone able to get it to work? I think it might have worked at time of writing but has since changed. How was the PR testeed?
but didn't manage to go through. I then tried the curl command:
and wasn't able to get a response. Tried
and was able to find the user though At first glance it looks like we currently we default the page params to a default value so not sure this branch will be reached Let me know if anyone has gotten the client lib bindings to work though |
Hey @J0, I tested this by running the auth service locally.
I rebuild the latest |
Thanks for the quick turnaround, let me check again and get back |
Curl command works thank you! I was looking at the wrong query my bad. For the auth-js bindings it generally seems great. Needed to change some parts of the response destructuring. Will take this to the team during our meeting in the next few hours and get back to everyone on next steps for this PR |
Hey as an update, We're going to close the PR in favour of a method like We'll revisit at a later that as the team is currently occupied with a few other tasks |
Feature request
Is your feature request related to a problem? Please describe.
On server-side, I want to check whether a user with a certain email exists before doing a signup. Since for safety reasons, signups on client side return fake "success" data, even though the user already signed up with this email. (#1517)
Describe the solution you'd like
I would like to have an additional method to not only list all users and get one by Id, but also get one by email.
So there is
supabase.auth.admin.getUserById(1)
but notgetUserByEmail('example@example.com')
Describe alternatives you've considered
listUsers()
, pretty inefficient but would do the trickThe text was updated successfully, but these errors were encountered: