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

configvalue = False is handled as True #390

Open
T0mWz opened this issue Feb 3, 2023 · 1 comment
Open

configvalue = False is handled as True #390

T0mWz opened this issue Feb 3, 2023 · 1 comment

Comments

@T0mWz
Copy link

T0mWz commented Feb 3, 2023

When a password_policy setting is set to False, it keeps handled as True.
Only when the configvalue is set to blank, then it is skipped.

+-----------------+---------------------------------------------------+-------------+
| appid           | configkey                                         | configvalue |
+-----------------+---------------------------------------------------+-------------+
| password_policy | spv_expiration_password_checked                   | false       |
| password_policy | spv_special_chars_checked                         | false       |
| password_policy | spv_user_password_expiration_checked              | false       |
| password_policy | spv_user_password_expiration_notification_checked | false       |

VS

+-----------------+---------------------------------------------------+-------------+
| appid           | configkey                                         | configvalue |
+-----------------+---------------------------------------------------+-------------+
| password_policy | spv_expiration_password_checked                   |             |
| password_policy | spv_special_chars_checked                         |             |
| password_policy | spv_user_password_expiration_checked              |             |
| password_policy | spv_user_password_expiration_notification_checked |             |
@prsnbrg
Copy link
Contributor

prsnbrg commented May 13, 2023

When you change the configuration via owncloud settings, the configvalue is set to blank for disabled requirements.
What is the benefit from your point of you if this behavior is changed?

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

No branches or pull requests

2 participants