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 should be required for user to change their account information #10122

Closed
brentgriffin opened this issue Nov 15, 2016 · 1 comment
Closed
Labels

Comments

@brentgriffin
Copy link
Contributor

Users are able to change account information including email and password without requiring their current password. So, if a user leaves dotCMS open or their session gets hijacked, their password and other account information can be changed without requiring the password to be reentered.

Expected Behavior

Any change on user account screen should require current password to be entered before the changes are persisted.
image

Steps to Reproduce (for bugs)

  1. Login to the dotCMS backend.
  2. Click your user name on the top right corner of the screen and select My Account from the menu.
  3. Enter the new password into the Password field.
  4. Re-enter the password into the Password Again field, making sure the password matches exactly.
  5. Press the Save button.
    Notice that everything is saved without requiring current password to be entered.

Context

Reported by client as a security vulnerability - https://my.dotcms.com/tickets/detail.dot?id=3b9e2a2d-3135-4a1c-b72f-d0a4d6332b94

Your Environment

  • dotCMS version used: 3.6 and eariler
@stale
Copy link

stale bot commented Sep 28, 2019

This issue has been automatically marked as stale because it has not had activity within the past 90 days. It will be closed in 30 days no further activity occurs. Thank you.

@stale stale bot added the wontfix label Sep 28, 2019
@stale stale bot closed this as completed Oct 28, 2019
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

1 participant