Page MenuHomePhabricator

Require MFA to edit MFA providers
ClosedPublic

Authored by epriestley on Jan 25 2019, 6:45 PM.

Details

Summary

Depends on D20037. Ref T13222. Ref T7667. Although administrators can now disable MFA from the web UI, at least require that they survive MFA gates to do so. T7667 (bin/auth lock) should provide a sturdier approach here in the long term.

Test Plan

Created and edited MFA providers, was prompted for MFA.

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Jan 25 2019, 6:45 PM
epriestley requested review of this revision.Jan 25 2019, 6:46 PM
amckinley accepted this revision.Jan 28 2019, 5:00 PM
This revision is now accepted and ready to land.Jan 28 2019, 5:00 PM
This revision was automatically updated to reflect the committed changes.