-
-
Notifications
You must be signed in to change notification settings - Fork 377
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
Kirbyrocket's suggestion: Display password in plain text back at a user via email #1296
Comments
Don't we echo back the password upon registration already? |
Yeah, this issue should be closed |
Thanks @DaBomberNTG. Since this complies with the original idea, it should be closed as @Simon311 previously said. Cc @nicatronTg |
@ProfessorXz you're right I'm an idiot |
@ProfessorXz @Simon311 @DaBomberNTG @Patrikkk please discuss this issue in the context of sending an email notification now not echoing back |
@nicatronTg As you said, this is a bad practice. Besides, Marcus already made a plugin for resetting password via e-mail, which should resolve the issue. |
@Simon311 did maru update it to the latest api / does it work in the latest api with |
@nicatronTg no idea |
@Simon311 ok |
Plugin is on latest api & initializes, if that is still relevant. |
@DaBomberNTG neat |
[Plaintext via email is not a best practice and a plugin accomplishes 99% of the goal easily] |
My two cents: this is easy to just drop in as a minor tweak/config option to echo the password back after registration. [Definitely not email though]. Echoing the password back is easy, emailing is bad/dangerous/a worst practice.
The text was updated successfully, but these errors were encountered: