Skip to content

Enable password reset for user with same email address when only one is active - #10743

Merged
blizzz merged 1 commit into
nextcloud:masterfrom
kesselb:bugfix/noid/allow-password-reset-for-duplicate-email
Sep 13, 2018
Merged

Enable password reset for user with same email address when only one is active#10743
blizzz merged 1 commit into
nextcloud:masterfrom
kesselb:bugfix/noid/allow-password-reset-for-duplicate-email

Conversation

@kesselb

Copy link
Copy Markdown
Contributor

When two or more user share the same email address its not possible to
reset password by email. Even when only one account is active.

This pr reduce list of users returned by getByEmail by disabled users.

…is active
When two or more user share the same email address its not possible to
reset password by email. Even when only one account is active.
This pr reduce list of users returned by getByEmail by disabled users.
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>

@nickvergessennickvergessen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me

@nickvergessen
nickvergessen requested review from blizzz and removed request for MorrisJobke and rullzerSeptember 13, 2018 07:15
@blizzz
blizzz merged commit ef97ef7 into nextcloud:masterSep 13, 2018
@kesselb
kesselb deleted the bugfix/noid/allow-password-reset-for-duplicate-email branch September 13, 2018 09:51
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@kesselb@nickvergessen@blizzz