-
Notifications
You must be signed in to change notification settings - Fork 6k
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
mgr/dashboard: Process password complexity checks immediately #32032
mgr/dashboard: Process password complexity checks immediately #32032
Conversation
8c05874
to
bb817a4
Compare
jenkins test make check |
bb817a4
to
096d19f
Compare
jenkins test dashboard |
jenkins test make check |
de65b6c
to
ba2b46d
Compare
jenkins test make check arm64 |
src/pybind/mgr/dashboard/frontend/src/app/core/auth/user-form/user-form.component.ts
Show resolved
Hide resolved
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.
Nice improvements :) I left some suggestions, maybe you could address the one or the other before your vacation starts ;)
src/pybind/mgr/dashboard/frontend/src/app/core/auth/user-form/user-form.component.ts
Show resolved
Hide resolved
src/pybind/mgr/dashboard/frontend/src/app/shared/api/user.service.ts
Outdated
Show resolved
Hide resolved
src/pybind/mgr/dashboard/frontend/src/app/shared/services/password-policy.service.ts
Outdated
Show resolved
Hide resolved
ba2b46d
to
ba3f46b
Compare
jenkins test dashboard backend |
jenkins test dashboard |
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.
LGTM :)
ba3f46b
to
979a63a
Compare
57ff167
to
1b6c539
Compare
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.
lgtm
jenkins test dashboard |
8969faf
to
f3ca3bf
Compare
jenkins test make check |
f3ca3bf
to
d29eb5a
Compare
jenkins test dashboard |
- Add 'api/user/validate_password' endpoint to check if a password meets the password policy. A new controller has to be added for 'api/user' which has NO security scope, otherwise it wouldn't be possible for users without USER privileges to call the endpoint. - Add Angular async validator to check if the entered password meets the policy. Fixes: https://tracker.ceph.com/issues/43088 Signed-off-by: Volker Theile <vtheile@suse.com>
1. Fix the test cases by using the 'assertJsonBody' method. The '_post' method doesn't return anything. That's why we need to use the 'assertJsonBody' method here instead of checking the fields directly. 2. Add the missing scope and permission definition of the 'validate_password' resource. Otherwise the resource is not restricted and 'test_validate_password_invalid_permissions' will fail. 3. Re-word error messages. Change 'cannot' to 'must not'. Signed-off-by: Tatjana Dehler <tdehler@suse.com>
d29eb5a
to
48de3dc
Compare
* Remove scope and permission check from UserPasswordPolicy controller again, otherwise the Angular UI validator can't be executed for users without that privilege. * Remobe obsolete QA test. Signed-off-by: Volker Theile <vtheile@suse.com>
48de3dc
to
bf567f4
Compare
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.
The QA test run was successful: http://pulpito.ceph.com/tdehler-2020-01-07_15:16:41-rados:dashboard-wip-tdehler-testing-32032-32234-32461-distro-basic-smithi/
jenkins test dashboard backend |
jenkins test dashboard |
jenkins test dashboard |
jenkins test dashboard backend |
Test failure:
|
Can not reproduce it on my local machine and the error does not look related to this PR, so i'll trigger another run. |
jenkins test dashboard backend |
Thanks for the update! |
jenkins test dashboard backend |
Fixes: https://tracker.ceph.com/issues/43088
Signed-off-by: Volker Theile vtheile@suse.com
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test dashboard backend
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox