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

Password reset on disabled user #28

Closed
alexander-schranz opened this issue Jul 22, 2016 · 3 comments · Fixed by #75
Closed

Password reset on disabled user #28

alexander-schranz opened this issue Jul 22, 2016 · 3 comments · Fixed by #75
Milestone

Comments

@alexander-schranz
Copy link
Member

alexander-schranz commented Jul 22, 2016

Q A
Bug? no
New Feature? no

Actual Behavior

You can reset a password of a user which is not enabled.

Expected Behavior

A. User can not disabled because he has not confirm his Mail Address
B. User can reset the password and is automatically enabled because he did confirm the email over the password reset link.

@alexander-schranz
Copy link
Member Author

I'm for B. what you think @wachterjohannes

@wachterjohannes
Copy link
Member

enabled because he has not confirmed his user? or disabled because of an "administrator"? in my opinion the user should not be able to send a password forget email if his user was not confirmed after registration.

@alexander-schranz alexander-schranz added this to the 1.0.0 milestone Aug 12, 2016
@alexander-schranz
Copy link
Member Author

alexander-schranz commented Dec 4, 2017

@wachterjohannes until it is not possible to resend the confirmation email (#37) I would go for B.

alexander-schranz added a commit to alexander-schranz/SuluCommunityBundle that referenced this issue Dec 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants