https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=25936

--- Comment #5 from Marcel de Rooy <[email protected]> ---
Great addition. Security++

Only thing I could say about the code:

if ( !C4::Auth::checkpw_hash( $password, $self_from_storage->password ) ) {

I understand this change but it should not really be bound to notify yes or no.
Could imagine that set_password should check that always and report it?
Currently the interface lets you change your password while not entering a new
one.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to