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

Log attempted user name on failed password reset attempts #33623

Merged
merged 2 commits into from Aug 22, 2022

Conversation

NoSleep82
Copy link
Contributor

i would be useful to know who is trying to reset the password (misspelled username or email, ex user or some sort of attack)

Signed-off-by: NoSleep82 52562874+NoSleep82@users.noreply.github.com

i would be useful to know who is trying to reset the password (misspelled username or email, ex user or some sort of attack)

Signed-off-by: NoSleep82 <52562874+NoSleep82@users.noreply.github.com>
@szaimen szaimen added enhancement 3. to review Waiting for reviews labels Aug 20, 2022
@szaimen szaimen added this to the Nextcloud 25 milestone Aug 20, 2022
@szaimen szaimen requested review from a team, blizzz, skjnldsv and CarlSchwan and removed request for a team August 20, 2022 11:25
core/Controller/LostController.php Outdated Show resolved Hide resolved
Co-authored-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
Signed-off-by: NoSleep82 <52562874+NoSleep82@users.noreply.github.com>
@blizzz blizzz merged commit fd645d4 into nextcloud:master Aug 22, 2022
@welcome
Copy link

welcome bot commented Aug 22, 2022

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

@blizzz blizzz changed the title Update LostController.php Log attempted user name on failed password reset attempts Aug 22, 2022
@blizzz blizzz mentioned this pull request Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants