data-govt-nz / ckanext-security

A CKAN extension to hold various security improvements for CKAN
GNU Affero General Public License v3.0
25 stars 32 forks source link

Fix invalid new password validation error #3

Closed catalyst-joe-guo closed 7 years ago

catalyst-joe-guo commented 7 years ago

Root reason:

ckanext.security.validators.user_password_validator raise a ValueError instead of a ckan.logic.ValidationError, ckan can not handle it in core.ckan.controllers.user._save_edit

Solution: Raise ValidationError with correct args then it can be handled.

External: WR277492