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

[Backend] Changing Customer Password - The warning message. #4119

Open
addison74 opened this issue Jul 25, 2024 · 2 comments
Open

[Backend] Changing Customer Password - The warning message. #4119

addison74 opened this issue Jul 25, 2024 · 2 comments
Labels

Comments

@addison74
Copy link
Contributor

addison74 commented Jul 25, 2024

Go to Backend > Customers > Manage Customers. Edit an existing customer then click on the Account Information tab. Let's change the password.

When you start to fill in the new password a warning is displayed. See bellow the screenshot.

screenshot-case1-1

Fill in the Current Admin Password and click the [Save and Continue Edit] button. The customer receives the message bellow

screenshot-case1-2

ISSUE
As can be seen from the warning message, the password is sent in plaintext format, but the customer does not receive it.

Here we have three variants to solve this issue

1. We send the password. For doing this, we have to change the email sent to the customer and ask him to reset the password immediately, because it was sent in plaintext format and it is not safe.

2. We do not send the password. We have to change the email sent to the customer and inform him that if the password was not set up by phone it will be communicated soon. This means that the administrator will contact him later by email or phone.

3. We give up setting the password by the administrator. We let only the implementation from PR #3262. The administrator will have the option to send the password reset link to the customer, nothing more. Below is how the implementation would look

screenshot-case1-3

Any feedback is welcome.

@addison74 addison74 added the bug label Jul 25, 2024
@hirale
Copy link
Contributor

hirale commented Jul 25, 2024

We give up setting the password by the administrator.

I vote for this.

@addison74
Copy link
Contributor Author

addison74 commented Jul 26, 2024

If the ability for the administrator to change the password of a customer account is removed, then the option from Backend > System > Configuration > Customer Configuration tab > Password Options section > "Require admin user to change user password" must also be removed too.

Screenshot 2024-07-26 155234

... or the functionality can be extended as follows. If "Require admin user to change user password" is set to "Yes" then the above behavior must be fixed. If it is set to "No", then the row with the password must be removed from the table.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants