]> 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)
committerLucas Gass <lucas@bywatersolutions.com>
Mon, 31 Oct 2022 17:17:23 +0000 (17:17 +0000)
commitb71531cd585b8b1288a57f47810659e31490f23c
tree837c004e2feb5bb4b10e01106b5f33bfedd1000c
parent5abc149682d346555b0393afb17221a4138aba86
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>
(cherry picked from commit edb16022febc4968d172d6e3e96fac295d31579d)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Koha/Auth/TwoFactorAuth.pm