Page MenuHomePhabricator
Differential D20036

Convert "Rename User" from session MFA to one-shot MFA
ClosedPublic
Actions

Authored by epriestley on Jan 25 2019, 6:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 8, 9:55 PM
Unknown Object (File)
Wed, May 7, 6:55 PM
Unknown Object (File)
Tue, May 6, 5:40 PM
Unknown Object (File)
Tue, May 6, 5:40 PM
Unknown Object (File)
Tue, May 6, 9:48 AM
Unknown Object (File)
Apr 21 2025, 2:34 PM
Unknown Object (File)
Apr 14 2025, 5:32 PM
Unknown Object (File)
Apr 11 2025, 7:02 PM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13222: 2018 Week 48-51 Bonus Content
Commits
rPd24e66724d3b: Convert "Rename User" from session MFA to one-shot MFA
Summary

Depends on D20035. Ref T13222.

  • Allow individual transactions to request one-shot MFA if available.
  • Make "change username" request MFA.
Test Plan
  • Renamed a user, got prompted for MFA, provided it.
  • Saw that I no longer remain in high-security after performing the edit.
  • Grepped for other uses of PhabricatorUserUsernameTransaction, found none.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
src/
applications/
people/
controller/
8 lines
xaction/
7 lines
transactions/
editor/
48 lines
storage/
6 lines

Diff 47865

View Options

src/applications/people/controller/PhabricatorPeopleRenameController.php

Loading...
View Options

src/applications/people/xaction/PhabricatorUserUsernameTransaction.php

Loading...
View Options

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...
View Options

src/applications/transactions/storage/PhabricatorModularTransactionType.php

Loading...
Phabricator · Built by Phacility