]> git.koha-community.org Git - koha.git/commit
Bug 32011: 2FA - Prevent qr_code to not be generated
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Thu, 27 Oct 2022 08:53:06 +0000 (10:53 +0200)
committerTomas Cohen Arazi <tomascohen@theke.io>
Thu, 27 Oct 2022 21:14:58 +0000 (18:14 -0300)
commitedb16022febc4968d172d6e3e96fac295d31579d
tree1c77d73cb7581ff83c57a40c7f3eb346c9631864
parentfd7bc8a9b8de5a8fdea36b046e4621760366dbf0
Bug 32011: 2FA - Prevent qr_code to not be generated

Caught by selenium/authentication_2fa.t

 #   Failed test 'Enforce 2FA setup on first login'
 #   at t/db_dependent/selenium/authentication_2fa.t line 291.Error while executing command: unexpected alert open: Dismissed user prompt dialog: [object Object] at /usr/share/perl5/Selenium/Remote/Driver.pm line 411.
 at /usr/share/perl5/Selenium/Remote/Driver.pm line 356.

There is a 500 in the logs:
 "POST /api/v1/app.pl/api/v1/auth/two-factor/registration HTTP/1.1" 500

That is caused by
[ERROR] POST /api/v1/auth/two-factor/registration: unhandled exception (Mojo::Exception)<<Overflow error. version 8
total bits: 1268  max bits: 1232>>

It's coming from GD::Barcode, in my understanding we should increase 'Version'.

Test plan:
Remove all other subtests from the selenium to speed up its exec, then
run it in a loop. Without this patch the tests fail quite often (1/10),
now it should not fail with this error (maybe another one, see bug
32010)

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Koha/Auth/TwoFactorAuth.pm