Skip to content
  • epriestley's avatar
    Allow objects to be put in an "MFA required for all interactions" mode, and... · d3c325c4
    epriestley authored
    Allow objects to be put in an "MFA required for all interactions" mode, and support "MFA required" statuses in Maniphest
    
    Summary:
    Depends on D19898. Ref T13222. See PHI873. Allow objects to opt into an "MFA is required for all edits" mode.
    
    Put tasks in this mode if they're in a status that specifies it is an `mfa` status.
    
    This is still a little rough for now:
    
      - There's no UI hint that you'll have to MFA. I'll likely add some hinting in a followup.
      - All edits currently require MFA, even subscribe/unsubscribe. We could maybe relax this if it's an issue.
    
    Test Plan:
      - Edited an MFA-required object via comments, edit forms, and most/all of the extensions. These prompted for MFA, then worked correctly.
      - Tried to edit via Conduit, failed with a reasonably comprehensible error.
    
    Reviewers: amckinley
    
    Reviewed By: amckinley
    
    Maniphest Tasks: T13222
    
    Differential Revision: https://secure.phabricator.com/D19899
    d3c325c4