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

refactor: align ui and backend password validation policy #2187

Merged

Conversation

leksinomi
Copy link
Contributor

@leksinomi leksinomi commented Oct 31, 2024

Description

This PR updates password validation to ensure consistency between frontend and backend components. Passwords are now required to be 12-50 characters long, including uppercase and lowercase letters, digits, and special characters, without spaces or consecutive repeating characters.

Related issues and discussion

#{Issue Number}

Screenshots, if any

Checklist

Please make sure these boxes are checked before submitting your pull request - thanks!

  • If you have multiple commits please combine them into one commit by squashing them.

  • Read and understood the contribution guidelines at web-app/.github/CONTRIBUTING.md.

@leksinomi leksinomi force-pushed the fix/enforce_stricter_password_policy branch from 7ffa1fe to 7751201 Compare October 31, 2024 22:35
@leksinomi leksinomi force-pushed the fix/enforce_stricter_password_policy branch from 7751201 to b0b14a0 Compare November 1, 2024 08:46
@leksinomi leksinomi changed the title fix: align ui and backend password validation policy refactor: align ui and backend password validation policy Nov 1, 2024
Copy link
Collaborator

@alberto-art3ch alberto-art3ch left a comment

Choose a reason for hiding this comment

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

LGTM

@alberto-art3ch alberto-art3ch merged commit 0762dfb into openMF:master Nov 4, 2024
1 of 2 checks passed
@leksinomi leksinomi deleted the fix/enforce_stricter_password_policy branch November 4, 2024 17:54
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