Page MenuHomePhabricator

Update a factor query in TransactionEditor for providers
ClosedPublic

Authored by epriestley on Wed, Jan 30, 11:05 PM.

Details

Summary

This query didn't get updated and could let you through an explicit "Sign with MFA" action if you have only disabled factors on your account.

Test Plan
  • Disabled all factors.
  • Used explicit "Sign With MFA".
    • Before: Went through.
    • After: Sensible error.

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.Wed, Jan 30, 11:05 PM
epriestley requested review of this revision.Wed, Jan 30, 11:06 PM
amckinley accepted this revision.Thu, Jan 31, 2:45 AM
This revision is now accepted and ready to land.Thu, Jan 31, 2:45 AM
This revision was automatically updated to reflect the committed changes.