The recovery codes form looks for the user input against a non-existent field.

The field's name is recovery, but the submit callback looks for the value at code. This patch changes the form field name to code.

CommentFileSizeAuthor
tfa-recovery-form-fix.patch602 bytesdaggerhart

Comments

daggerhart created an issue. See original summary.

daggerhart’s picture

daggerhart commented
Status: Active » Needs review
therealssj’s picture

therealssj commented

Applies cleanly against latest branch.
This will come into action when we start supporting multiple TFA as proposed by this patch #2923786: Allow users to select from multiple 2nd-Factor approaches.
Let's get this merged.

therealssj’s picture

therealssj commented
Status: Needs review » Reviewed & tested by the community

  • nerdstein committed b2cd086 on 8.x-1.x authored by daggerhart 
    Issue #2924970 by daggerhart: Recovery code form broken- checking for...
nerdstein’s picture

nerdstein
he/him
Primary language English
Location United States
commented
Status: Reviewed & tested by the community » Fixed

This is fixed, thanks for all the good work both of you

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.