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

🐛 [FirebaseAuth] "sendPasswordResetEmail" method doesn't catch exceptions #11739

Closed
TheStrongest opened this issue Oct 19, 2023 · 1 comment
Labels
resolution: duplicate This issue or pull request already exists type: bug Something isn't working

Comments

@TheStrongest
Copy link

Bug report

Describe the bug
When testing my authentication flow I found out that the "sendPasswordResetEmail" method does not catch/throw exceptions. I tried using different fake emails that are not registered on my database to test it, but no error was thrown in the end. Instead, the try/catch block was passed successfully.


@TheStrongest TheStrongest added Needs Attention This issue needs maintainer attention. type: bug Something isn't working labels Oct 19, 2023
@darshankawar darshankawar added the triage Issue is currently being triaged. label Oct 20, 2023
@darshankawar
Copy link

@TheStrongest
The case for sendPasswordResetEmail is mentioned in this open issue: #3273
Please follow-up in it for further updates. Closing as duplicate.

@darshankawar darshankawar closed this as not planned Won't fix, can't repro, duplicate, stale Oct 20, 2023
@darshankawar darshankawar added resolution: duplicate This issue or pull request already exists and removed Needs Attention This issue needs maintainer attention. triage Issue is currently being triaged. labels Oct 20, 2023
@firebase firebase locked and limited conversation to collaborators Nov 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
resolution: duplicate This issue or pull request already exists type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants