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

🚀 Feature: Share password policy #696

Open
chileocat opened this issue Dec 3, 2024 · 4 comments
Open

🚀 Feature: Share password policy #696

chileocat opened this issue Dec 3, 2024 · 4 comments
Labels
feature New feature or request

Comments

@chileocat
Copy link

🔖 Feature description

It would be really nice, if you could define password policy. For the beginn, also setting the min. password lenght to 8 would be awesome.

🎤 Pitch

We're using pingvin in our company for transfering different kind of documents and images. And some lazy users just setting 123 or aaa as password. :-)

@chileocat chileocat added the feature New feature or request label Dec 3, 2024
@stonith404
Copy link
Owner

The requirement is that the password must be at least 8 chars. Are you sure that this isn't validated correctly?

@chileocat
Copy link
Author

So you mean, it should already at least 8 chars? Just checked, with only 3 it's ok.
grafik

@stonith404
Copy link
Owner

@chileocat Oh you mean the share password. I though you mean the user password. I keep this open then.

@stonith404 stonith404 changed the title 🚀 Feature: Password policy 🚀 Feature: Share password policy Dec 3, 2024
@aarondoet
Copy link
Contributor

Password requirements are a complex topic because there are many variables. Require a minimum length, require uppercase/lowercase characters, require digits, require special symbols, disallow common passwords etc.

I think adding too many options for this is not really useful. My ideas currently:

  1. minimum share password length - nothing more and nothing less
  2. toggle to require secure passwords - either there are no requirements (I would even lower the min password length to 1) or you need at least one uppercase and lowercase character, a digit, maybe a special symbol, and minimum password length is 8

Also maybe a toggle to require shares to have passwords?

What do you guys think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants