Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.It would be great if there was a setting to force authenticated users to enable two factor authentication after logging in if they haven't already.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | interdiff_2-6.txt | 1.08 KB | el7cosmos |
| #6 | force-otp-setting-3074465-6.patch | 9.25 KB | el7cosmos |
Issue fork one_time_password-3074465
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
eric115 commentedPatch attached
Comment #3
studiosander commentedI tested this patch against the dev version. Works fine... Please apply this patch to the RC version... or make me co-owner so I can apply and maintain this module!
Greets,
Sander
Comment #4
sam152 commentedI think this belongs in it's own contrib. I'd be happy to link it from the project page.
Comment #5
dpi@studiosander Its safe to say the project remains well maintained, however this isnt the kind of direction the project wants to go in.
Comment #6
el7cosmos\Symfony\Component\HttpKernel\Event\GetResponseEventis deprecated since Symfony 4.3 and is no longer available in Symfony 6.2. As Drupal 10 uses Symfony 6.2, this must be changed to use\Symfony\Component\HttpKernel\Event\RequestEventinsteadComment #9
mstrelan commentedDiscussed with @larowlan who agreed to reopen this. Rerolled the patch in #6, added some constructor property promotion, autowire, autoconfigure and return types. Needs work to get the pipeline green.
Comment #10
mstrelan commentedThat was more involved than I had hoped. Should be green now.
Comment #13
larowlanSpoke with @dpi about this.
I realise @sam152 was keen to keep this module lean, but with the latest release that includes a fair bit on the security front, that ship has sailed.
We're running this patch on several sites so rather than continually rerolling it, going to merge it.
Comment #15
larowlanFor those wishing to remain on the branch without this, I've put it into a 2.x branch
Comment #16
larowlanThis will only be in the 2.x series