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

--- Comment #7 from David Cook <[email protected]> ---
(In reply to Jonathan Druart from comment #6)
> Why don't you set it in Koha/Auth/TwoFactorAuth.pm to also affect the call
> from Koha/REST/V1/TwoFactorAuth.pm (used for the registration)?

Funny you mention it... I did write a version where I overrode verify() but
then I thought maybe we wouldn't want to embed C4::Context in
Koha/Auth/TwoFactorAuth.pm, so I opted for this version instead. 

Both options seemed to have their pros and cons, and I was fine with both
really. 

Happy to do the override version instead if folk prefer that.

-- 
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