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
F14041000: D9285.id22050.diff
Mon, Nov 11, 2:58 PM
F14040999: D9285.id22049.diff
Mon, Nov 11, 2:58 PM
F14040998: D9285.id22319.diff
Mon, Nov 11, 2:58 PM
F14035846: D9285.id22319.diff
Sun, Nov 10, 7:46 AM
F14034769: D9285.id22049.diff
Sun, Nov 10, 2:06 AM
F14013405: D9285.id22319.diff
Sat, Nov 2, 6:45 AM
F14002890: D9285.diff
Fri, Oct 25, 11:39 PM
F13993179: D9285.id22049.diff
Tue, Oct 22, 9:12 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
Lint
Lint Skipped
Unit
Tests Skipped

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