Page MenuHomePhabricator

Allow installs to require multi-factor authentication for all users
ClosedPublic

Authored by epriestley on May 25 2014, 12:11 AM.
Tags
None
Referenced Files
F13246085: D9285.id22050.diff
Thu, May 23, 7:34 AM
F13216489: D9285.id.diff
Sat, May 18, 1:11 AM
Unknown Object (File)
Mon, Apr 29, 7:52 PM
Unknown Object (File)
Sat, Apr 27, 5:21 PM
Unknown Object (File)
Apr 21 2024, 11:23 PM
Unknown Object (File)
Apr 21 2024, 11:23 PM
Unknown Object (File)
Apr 21 2024, 11:23 PM
Unknown Object (File)
Apr 21 2024, 10:42 PM
Subscribers

Details

Summary

Ref T5089. Adds a security.require-multi-factor-auth which forces all users to enroll in MFA before they can use their accounts.

Test Plan

Config:

Screen_Shot_2014-05-24_at_5.05.39_PM.png (897×1 px, 133 KB)

Roadblock:

Screen_Shot_2014-05-24_at_5.04.58_PM.png (986×1 px, 132 KB)

After configuration:

Screen_Shot_2014-05-24_at_5.05.17_PM.png (986×1 px, 127 KB)

  • Required MFA, got roadblocked, added MFA, got unblocked.
  • Removed MFA, got blocked again.
  • Used bin/auth strip to strip MFA, got blocked.

Diff Detail

Repository
rP Phabricator
Branch
rauth2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 692
Build 692: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Allow installs to require multi-factor authentication for all users.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jun 3 2014, 9:55 PM
epriestley updated this revision to Diff 22319.

Closed by commit rP99c72a32d041 (authored by @epriestley).