6 Sep
2023
6 Sep
'23
9:18 a.m.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30843 --- Comment #7 from David Cook <dcook@prosentient.com.au> --- (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.