Skip to content
This repository has been archived by the owner on May 3, 2020. It is now read-only.

Adding password complexity checking for local account authentication. #562

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

Conversation

ghost
Copy link

@ghost ghost commented May 26, 2019

I saw the comment about the TODO for password complexity and I needed it for my server. Happy to contribute! Hope ya'll like it.

@MaxNad
Copy link
Collaborator

MaxNad commented Jul 5, 2019

Hi @schlpr0k ,
Thank you for contributing. I really like the improvement idea.

There is one thing I would like to change before merging this into the project. The only problem I see with that current change is the lack of flexibility in the password filtering. For instance, somebody might want to force 15 chars instead of 12 or maybe require 50 chars but not complexity.

I will try to see how we could build a bit more flexibility on top of your change and merge it then.

Thanks again a lot for the contribution.

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

Successfully merging this pull request may close these issues.

None yet

1 participant