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

Event indicating that the confirmation of updateUser was successful #1560

Open
yegor-pelykh opened this issue Apr 27, 2024 · 2 comments
Open
Labels
auth enhancement New feature or request

Comments

@yegor-pelykh
Copy link

Why, after a user successfully confirms an email address change (after calling updateUser), the application doesn't receive an event confirming this action?
The application should receive a userUpdated event or some new type of event.
Otherwise, it is now impossible to know that the confirmation was successful.

To Reproduce

  1. call auth.updateUser
  2. click on the confirmation link in the email

Expected behavior

Application should receive an event indicating that the confirmation was successful.

@yegor-pelykh yegor-pelykh added the bug Something isn't working label Apr 27, 2024
@encima encima added enhancement New feature or request auth and removed bug Something isn't working labels Apr 29, 2024
@encima
Copy link

encima commented Apr 29, 2024

Thanks for opening!
updateUser can indicate changes beyond only the email changing and those are typically instant (i.e. updating the password or metadata).

Would you want this event to trigger for all possibilities of updateUser or are you looking for an event for something like email/phone changed?

Note: I am moving this to the Auth repo so the right team is involved

@encima encima pinned this issue Apr 29, 2024
@encima encima unpinned this issue Apr 29, 2024
@encima encima transferred this issue from supabase/supabase Apr 29, 2024
@yegor-pelykh
Copy link
Author

Hi @encima, thanks for the answer!

Would you want this event to trigger for all possibilities of updateUser or are you looking for an event for something like email/phone changed?

I would say, it would be logical to provide this event for all cases where some kind of user action outside the application is required. For example, as in my case, confirmation by clicking on a link in an email.
I would prefer to see another raising of userUpdated event for each such action outside the application, and along with the event provide an actual session with user data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auth enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants