Skip to content

[Feature] Reset password CLI command - #3491

Merged
carakas merged 3 commits into
forkcms:masterfrom
SavageTiger:master
Jul 31, 2023
Merged

carakas merged 3 commits into
forkcms:masterfrom
SavageTiger:master

Conversation

@SavageTiger

Copy link
Copy Markdown
Contributor

Type

  • Feature / Enhancement

Pull request description

I maintain a Website that runs on ForkCMS, it commonly happens that I get a call that somebody forgot their password since they do not use it on a regular basis.

I know there is a reset password feature, but that requires you to have access to an email-box, and as the server admin I would just like to help out the person right away.

So I developed this CLI command to reset the password using the Symfony Console, and since it's so useful to me, I though maybe this is also useful to other site maintainers.

Since I want to add the CLI to reset a password the the Module it belong to we require a DI service loader extension.
@SavageTiger
SavageTiger requested a review from a team as a code owner January 20, 2022 10:38

@carakas carakas 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.

Some small things you should check but overall cool feature
Thank you for your contribution

Comment thread src/Backend/Modules/Users/Console/ResetPasswordCommand.php Outdated
Comment thread src/Backend/Modules/Users/Resources/config/services.yml Outdated
In order to reset a password you need access to a mailbox, however the server might want to be able to reset the password for a user using his / her access to the server itself.
@SavageTiger

Copy link
Copy Markdown
Contributor Author

@carakas Done and Done

@carakas
carakas merged commit 34c1c4c into forkcms:master Jul 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants