-
Notifications
You must be signed in to change notification settings - Fork 27
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
improve next #410
improve next #410
Conversation
* add userlog_global_notifications_secret * add secrets for userlog * add secrets for userlog * add secret options * fix notifications secrets * fix notifications secrets * fix notifications secrets * add docs
8f02723
to
46e2bdd
Compare
# the path to the file containing the list of banned passwords | ||
bannedPasswordsList: "" | ||
# list of banned passwords | ||
bannedPasswords: [] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if this will be sufficient in the future. I heard that there will be thousands of passwords in this banned password list. But we can iterate over this later
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know where this should go. If we have thousands of passwords we might hit the ConfigMap size limit quite quickly.
But to be honest: is a list of thousands of passwords that is regularly updated still application configuration? I would rather thread this like eg. ClamAV virus signature databases... That need to be fetch from some sort of mirror
Description
improve
next
branchRelated Issue
next
branch tracker for oCIS latest 5.0 compatiblity #406minCharacters
settingMotivation and Context
improve
next
How Has This Been Tested?
bannedPasswords
and tested if they are enforcedScreenshots (if appropriate):
Types of changes
Checklist:
make docs
) and committed