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
F14948379: D9285.diff
Sat, Feb 15, 9:26 PM
Unknown Object (File)
Sun, Feb 9, 1:10 PM
Unknown Object (File)
Sun, Feb 9, 1:10 PM
Unknown Object (File)
Sun, Feb 9, 1:10 PM
Unknown Object (File)
Sun, Feb 9, 1:10 PM
Unknown Object (File)
Sun, Feb 2, 12:24 PM
Unknown Object (File)
Thu, Jan 30, 4:39 PM
Unknown Object (File)
Mon, Jan 27, 4:45 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 693
Build 693: [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).