1199090 - add printable recovery 2fa codes

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: General, defect, P2)

Description

[:glob ✱]

add printable recovery 2fa codes

Comment 1

[:glob ✱]

Attachment: 1199090_1.patch

- add ability to generate codes
- allows codes to be used anywhere 2fa is required
- minor fixes (uninit warnings, printing tweaks)

Comment 2

[Dylan Hardison [:dylan] (he/him)]

Comment on attachment 8664662 [details] [diff] [review]
1199090_1.patch

Review of attachment 8664662 [details] [diff] [review]:
-----------------------------------------------------------------

r- because inconsistent calling convention of verify_check() -- in one place it is called with no arguments,
and in another with arguments. The behavior seems fine but I'd like that to be fixed before I finish reviewing it.

::: Bugzilla/MFA.pm
@@ +90,5 @@
> +}
> +
> +# methods
> +
> +sub generate_recovery_codes {

nit 1: I don't think we really need three loops here. 
nit 2: the first loop doesn't conform to our usual coding conventions.

::: userprefs.cgi
@@ +675,5 @@
>          clear_settings_cache(Bugzilla->user->id);
>      }
> +
> +    elsif ($action eq 'recovery') {
> +        $user->mfa_provider->verify_check(Bugzilla->input_params);

Argument passed to method is not used.

@@ +686,2 @@
>      else {
> +        $user->mfa_provider->verify_check(Bugzilla->input_params);

Argument passed to method is not used by method.

Comment 3

[:glob ✱]

Attachment: 1199090_2.patch

- actually use the passed param
- simplify code generation

Comment 4

[Dylan Hardison [:dylan] (he/him)]

Comment on attachment 8665259 [details] [diff] [review]
1199090_2.patch

Review of attachment 8665259 [details] [diff] [review]:
-----------------------------------------------------------------

r=dylan

Comment 5

[:glob ✱]

To ssh://[e-mail]/webtools/bmo/bugzilla.git
   87c32cb..05fed61  master -> master