Add option to force authenticated users to enable OTP

Created on 13 August 2019, almost 6 years ago
Updated 14 May 2025, 2 days ago

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.

Feature request
Status

Closed: won't fix

Version

1.0

Component

Code

Created by

🇦🇺Australia Eric115

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • 🇮🇩Indonesia el7cosmos 🇮🇩 GMT+7

    \Symfony\Component\HttpKernel\Event\GetResponseEvent is 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\RequestEvent instead

  • First commit to issue fork.
  • Merge request !103074465 → (Merged) created by mstrelan
  • Pipeline finished with Failed
    2 days ago
    Total: 227s
    #497585
  • Pipeline finished with Failed
    2 days ago
    Total: 149s
    #497588
  • 🇦🇺Australia mstrelan

    Discussed 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.

  • Pipeline finished with Failed
    2 days ago
    Total: 151s
    #497600
  • Pipeline finished with Failed
    2 days ago
    Total: 189s
    #497610
  • Pipeline finished with Failed
    2 days ago
    Total: 241s
    #497622
  • Pipeline finished with Success
    2 days ago
    Total: 319s
    #497629
  • Pipeline finished with Running
    2 days ago
    #497631
  • 🇦🇺Australia mstrelan

    That was more involved than I had hoped. Should be green now.

  • Pipeline finished with Skipped
    2 days ago
    #497661
  • First commit to issue fork.
  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    Spoke 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.

    • larowlan committed 37a3c61b on 8.x-1.x
      Revert "Issue #3074465 by mstrelan, el7cosmos, eric115, sam152, dpi: Add...
  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    For those wishing to remain on the branch without this, I've put it into a 2.x branch

  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    This will only be in the 2.x series

Production build 0.71.5 2024