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

ASVS 2.1 Password Security Requirements- Change/Reset Password for user with emergency access #7350

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RatishkumarS
Copy link

@RatishkumarS RatishkumarS commented Apr 14, 2024

Fixes #7349

Short description of what this resolves:

This enhances password Security Requirements for a user who tries to login and if he/she forget their credentials and they have an emergency to enter the application. In these cases there should be an easy and effective mechanism to get their access. As OpenEMR is an health related application. It should be focusing on user friendly authorization

Screenshot 2024-04-14 at 7 40 53 PM Screenshot 2024-04-14 at 7 41 01 PM Screenshot 2024-04-14 at 7 41 11 PM

Changes proposed in this pull request:

Change password functionality for existing users.
Update old password with new password in the myphpAdmin database.

@adunsulag
Copy link
Sponsor Member

A couple of the administrators had a discussion about this in the past. Please review the following OpenEMR community forum thread about implementing this feature.

https://community.open-emr.org/t/forget-password-in-login-screen/8899/13

I'm fine with a forgot password option but it needs to be configurable by a global setting as there are some installations (as discussed in the thread) that will turn this functionality off.

Also, I would like to hear from you on the discussion in the forum thread of the need or lack of need for MFA to be turned on in order for reset password to be enabled. I think a compromise would be to have another global option to require MFA for all password resets (turned on by default) and to allow installations to turn this off. If you need help on how to add a global setting please let us know and we can provide guidance.

@adunsulag adunsulag added the WaitingForInfo This will put in queue for stale bot label Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WaitingForInfo This will put in queue for stale bot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change/Reset Password for users with emergency access
2 participants