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

LNP-641_Fresh installation and configuration of Password Policy module #639

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

Conversation

sweetymj
Copy link
Contributor

@sweetymj sweetymj commented Jan 6, 2025

Context

Does this issue have a Jira ticket?
LNP-641
If this is an issue, do we have steps to reproduce?
N/A

Intent

What changes are introduced by this PR that correspond to the above card?

Fresh installation and configuration of the Password Policy module.

  • Installed the Password Policy module to enforce strong password standards.
  • Configured the following password constraints:
    • Minimum password length: 8
    • Minimum password character types: 4 (The password must include one uppercase letter, one lowercase letter, one number and one special character)
    • Password must not contain the user's username
  • Set password reset days to 30
  • Enabled "Send email upon password expiration'
  • Set pending email days to 7, 14, 30.

Would this PR benefit from screenshots?
N/A

Considerations

Is there any additional information that would help when reviewing this PR?
N/A
Are there any steps required when merging/deploying this PR?
N/A

Checklist

  • This PR contains only changes related to the above card
  • This deployment has been tested for cache invalidation
  • Tests have been added/updated to cover the change
  • Documentation has been updated where appropriate
  • Tested in Development

@sweetymj sweetymj requested a review from Eli-TW January 6, 2025 17:59
Copy link
Contributor

@Eli-TW Eli-TW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mandating password expiry is against guidance: https://security-guidance.service.justice.gov.uk/passwords/#app-based-password-protection-for-files

Also - I don't think this is complete; I deployed to dev and successfully set an insecure password. Do we need to specify all the roles against the policy for this to be enforce?

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